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

EPP 160 Precursor details page #73

Open
wants to merge 23 commits into
base: master
Choose a base branch
from

Conversation

vivekkumar-ho
Copy link
Contributor

@vivekkumar-ho vivekkumar-ho commented Feb 13, 2025

What?

EPP-160 - PRECURSOR DETAILS page

Why?

PRECURSOR DETAILS page is created to users can select the details for the selected precursor

How?

  • Created relevant fields and labels
  • Helpers for custom validation
  • behaviour for store and display the home address

Testing?

Local and branch testing

Screenshots (optional)

Anything Else? (optional)

Check list

  • I have reviewed my own pull request for linting issues (e.g. adding new lines)
  • I have written tests (if relevant)
  • I have created a JIRA number for my branch
  • I have created a JIRA number for my commit
  • I have followed the chris beams method for my commit https://cbea.ms/git-commit/
    here is an example commit
  • Ensure drone builds are green especially tests
  • I will squash the commits before merging

@vivekkumar-ho vivekkumar-ho changed the title WIP: Epp 160 precursor details page WIP: EPP 160 Precursor details page Feb 18, 2025
@vivekkumar-ho vivekkumar-ho changed the title WIP: EPP 160 Precursor details page EPP 160 Precursor details page Feb 19, 2025
@PaolaDMadd-Pro
Copy link
Contributor

The work looks fine to me. But I couldn't find any information regarding the confirm summary page. I know that this section will have a mid-summary in the flow.
Can you please update the Jira ticket as well as this PR for track purposes?
thank you

@vivekkumar-ho
Copy link
Contributor Author

Sure, thats fine. As per the figma and JIRA, next route is the /precursors-summary. It’s a common functionality that wherever we have a section summary for any specific page, section summary will be added to the check your answer page and not the page itself.

I’ve added the comment in the JIRA. Any other specific information you want me to add?

Copy link
Contributor

@PaolaDMadd-Pro PaolaDMadd-Pro left a comment

Choose a reason for hiding this comment

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

looks good to me

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