Skip to content

PM-6555: Show the app drawer for administrators - #420

Merged
jmgasper merged 1 commit into
devfrom
fix/PM-6555-admin-app-drawer
Oct 13, 2026
Merged

jmgasper merged 1 commit into
devfrom
fix/PM-6555-admin-app-drawer

Conversation

@jmgasper

@jmgasper jmgasper commented Oct 13, 2026 •

Copy link
Copy Markdown
Collaborator

Fixes administrator app drawer visibility reported in PM-6555.

UniNav discarded the administrator JWT role because it was missing from its recognized roles. An administrator who also had Topcoder Customer was consequently treated as customer-only: both drawer sections were filtered out and the trigger added in #419 was hidden. An administrator without a recognized navigation role received only the fallback Talent section.

Recognize administrator (including legacy capitalization such as Administrator) and include it in the Talent and Staff guards. Administrators now receive both sections even when they also have Customer or Talent roles. Customer-only accounts still have no drawer, and application-specific roles such as Payment Admin do not grant Staff navigation. Document the role rules and regenerate TypeScript declarations. Linked applications continue enforcing their own permissions.

Validation:

  • Four new administrator regression cases failed before the fix and pass afterward; all 10 tests pass with npm test.
  • npm run lint: 0 errors, 0 warnings.
  • npm run build: main and nudge production bundles plus declarations succeed; browser validation uses the checked-in development environment configuration.
  • Headless Chromium smoke checks of the built marketing navigation pass for seven account scenarios at 1440px and 390px (14 cases): admin, admin + customer, legacy admin + customer, staff, talent, customer-only, and signed out. Verified drawer opening, mobile Staff scrolling, and closing.

Browser checks use synthetic JWT/profile fixtures, not mess's live session. QA should retest mess on topcoder-dev.com after deployment. Targets dev, confirmed as this repository's development branch.

@jmgasper
jmgasper merged commit 7880433 into dev Oct 13, 2026
6 of 7 checks passed
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