Skip to content

Feat(policy): Add PDF upload capability to Policy Manager#3694

Closed
Br0wnHammer wants to merge 2 commits intodevelopfrom
feat/pdf-policy-manager
Closed

Feat(policy): Add PDF upload capability to Policy Manager#3694
Br0wnHammer wants to merge 2 commits intodevelopfrom
feat/pdf-policy-manager

Conversation

@Br0wnHammer
Copy link
Copy Markdown
Member

Describe your changes

  • Add PolicyAttachments component for attaching PDF files to policies using existing FileManagerUploadModal and FilePickerModal
  • Integrate PDF attachments into PolicyEditorPage (visible for existing policies, below metadata form)
  • Fix file_entity_links unique constraint to include organization_id, aligning with the ON CONFLICT clause in file.repository.ts

Please ensure all items are checked off before requesting a review:

  • I deployed the code locally.
  • I have performed a self-review of my code.
  • I have included the issue # in the PR.
  • I have labelled the PR correctly.
  • The issue I am working on is assigned to me.
  • I have avoided using hardcoded values to ensure scalability and maintain consistency across the application.
  • I have ensured that font sizes, color choices, and other UI elements are referenced from the theme.
  • My pull request is focused and addresses a single, specific feature.
  • If there are UI changes, I have attached a screenshot or video to this PR.
Screenshot 2026-04-09 at 00 24 27

@Br0wnHammer Br0wnHammer added this to the 2.3 milestone Apr 8, 2026
@Br0wnHammer Br0wnHammer requested a review from gorkem-bwl April 8, 2026 18:56
@Br0wnHammer Br0wnHammer self-assigned this Apr 8, 2026
@Br0wnHammer Br0wnHammer added the enhancement New feature or request label Apr 8, 2026
@gorkem-bwl gorkem-bwl closed this Apr 9, 2026
@gorkem-bwl gorkem-bwl deleted the feat/pdf-policy-manager branch April 9, 2026 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants