Skip to content

refactor(web): extract apps page helpers - #11

Merged
thomasorgeval merged 1 commit into
mainfrom
small-refactor/apps-page-helpers
Jun 22, 2026
Merged

refactor(web): extract apps page helpers#11
thomasorgeval merged 1 commit into
mainfrom
small-refactor/apps-page-helpers

Conversation

@thomasorgeval

Copy link
Copy Markdown
Member

Summary

  • Extracted repeated Apps page active-member role loading into a single helper.
  • Extracted shared Connect App dialog configuration used by create and edit flows.
  • Behavior is intended to be unchanged.

Expected impact

  • Keeps the Apps page dialog setup and role refresh behavior in one place, reducing duplication and review surface for future tweaks.
  • Makes create/edit dialog differences explicit while preserving their existing result handling.

Verification

  • pnpm format:check apps/web/src/app/pages/apps/apps.page.ts
  • pnpm nx typecheck web --skip-nx-cache --tui=false
  • pnpm lint apps/web/src/app/pages/apps/apps.page.ts
  • git diff --check main..HEAD

@thomasorgeval
thomasorgeval marked this pull request as ready for review June 22, 2026 08:24
@thomasorgeval
thomasorgeval merged commit f038fac into main Jun 22, 2026
1 check passed
@thomasorgeval
thomasorgeval deleted the small-refactor/apps-page-helpers branch June 22, 2026 08:25
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