Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(APG-456c) Show POM details below Referrer details #820

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jsrobertson
Copy link
Contributor

Context

As part of ‘referral details’ in both the POM and PT journeys we need to display who the primary POM is, along with their email address.

Changes in this PR

Updated ShowReferralUtils.submissionSummaryListRows to return POM details.

Screenshots of UI changes

Before

image image

Release checklist

Release process documentation

As part of our continuous deployment strategy we must ensure that this work is
ready to be released once merged.

Pre-merge

  • There are changes required to the Accredited Programmes API for this change to work...
    • ... and they have been released to production already

Post-merge

  • This adds, extends or meaningfully modifies a feature...
  • This makes new expectations of the API...
    • ... and I have notified the API developer(s) of changes to the contract tests (Pact), or the API is already compliant
  • Manually approve release to preprod
  • Manually approve release to prod

@jsrobertson
Copy link
Contributor Author

Not merging until all referrals have been updated with POM data.

@jsrobertson jsrobertson force-pushed the APG-456-c-show-pom-details branch from 4a099cb to f3fb927 Compare December 18, 2024 10:06
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.

2 participants