Skip to content

PHEE-401: token refresh, G2P mock, Account Mapper rebuild, CI workflow and TS fixes - #14

Merged
DavidH-1 merged 2 commits into
openMF:devfrom
Flashl3opard:feat/token-refresh-msw-docs-g2p-mock
Aug 19, 2026
Merged

DavidH-1 merged 2 commits into
openMF:devfrom
Flashl3opard:feat/token-refresh-msw-docs-g2p-mock

Conversation

@Flashl3opard

@Flashl3opard Flashl3opard commented Aug 19, 2026

Copy link
Copy Markdown
Member

Related

Epic: PHEE-363
Closes: PHEE-401

Summary

Adds session management, G2P Config MSW mock, Account Mapper
Register/Update workflow, CI pipeline and TypeScript build fixes.

Changes

Session Management & Token Refresh

  • 60s polling checks JWT expiry in KeycloakProvider.tsx
  • Proper Base64URL decoding (fixes valid token false-logout bug)
  • Warning toast shown 2 minutes before session expires
  • Auto logout and redirect to /login on token expiry
  • 401 Axios interceptor clears token and redirects to /login
  • Shared ToastProvider context for app-wide notifications

Account Mapper — Register/Update Workflow

  • CreateBeneficiariesTab.tsx fully rebuilt with Register/Update mode toggle
  • Dynamic multi-beneficiary form (add/remove entries)
  • Fields: Payee Identity, Payment Modality (00-04), BBIC, Financial Address
  • Source BBID field in Update mode
  • Collapsible API Config section (API URL, Callback URL, Institution ID)
  • Form validation with error banner
  • SelfServicePortal.tsx updated with read-only → Edit → Save/Cancel flow
  • BBIC added to Beneficiary type and mock data

G2P Config Mock (MSW)

  • MSW handlers for g2pPaymentConfig, governmentEntity, program, dfsp
  • Dual dev modes: npm run dev (Gazelle) vs npm run dev:g2p (MSW mock)

CI/CD Pipeline

  • Added .github/workflows/ci.yml
  • Runs npm ci, npm run build, tsc --noEmit on push/PR to dev and main

TypeScript & Build Fixes (from Giulio's review)

  • Removed deprecated baseUrl from both tsconfigs (fixes TS5101)
  • Typed fetchMainBatches return type (fixes implicit-any cascade)
  • Fixed unused variables in UserManagementTab and Dashboard
  • npm audit fix — 0 vulnerabilities

Filter Chips

  • Main Batches filter chips now correctly filter real API uppercase
    status strings (COMPLETED, IN_PROGRESS, FAILED)

Screenshots

Screenshot 2026-08-19 162349 Screenshot 2026-08-19 162334

Notes

  • Account Mapper APIs are mock for now (backend pending)
  • Chrome required for Gazelle SSL cert acceptance

@Flashl3opard
Flashl3opard requested review from a team and a lite review from Copilot August 19, 2026 10:57

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@DavidH-1

Copy link
Copy Markdown
Contributor

Why are account mapper APIs mock?

@DavidH-1 DavidH-1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm my comments can be addressed in a subsequent pr

@DavidH-1
DavidH-1 merged commit 4864066 into openMF:dev Aug 19, 2026
2 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.

3 participants