Skip to content

Feature/consistency etc#162

Open
sei-tspencer wants to merge 6 commits intomainfrom
feature/consistency-etc
Open

Feature/consistency etc#162
sei-tspencer wants to merge 6 commits intomainfrom
feature/consistency-etc

Conversation

@sei-tspencer
Copy link
Copy Markdown
Contributor

Permission fixes:

  • Broaden collection loading to account for Edit/Manage permissions,
    not just ViewCollections
  • Fix race condition by moving collection loading to ngOnInit
  • Show admin sidebar tabs for users with Edit or Manage permissions
  • Gate exhibit create on CreateExhibits system permission
  • Separate Edit and Manage permission checks so Manage no longer
    implies Edit for collections and exhibits
  • Gate exhibit delete on Manage permission instead of Edit
  • Deduplicate card list to fix NG0955 tracking error after sharing

Advance button:

  • Add move/inject display and advance button to the wall view
  • Wire up advanceExhibit API call via ExhibitDataService
  • Show advance button only for users who can manage the exhibit
    and when the exhibit has showAdvanceButton enabled

Article styling:

  • Apply theme background to article container
  • Prevent theme colors and fonts from bleeding into angular-editor
    and iframe content in article and article-more-dialog components
  • Make article page div scrollable, hide scrollbar when printing

@sei-tspencer sei-tspencer requested a review from a team as a code owner April 10, 2026 20:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant