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

Feature/aapd 951 #3429

Merged
merged 18 commits into from
Feb 15, 2024
Merged

Feature/aapd 951 #3429

merged 18 commits into from
Feb 15, 2024

Conversation

Si3rr4wow
Copy link
Collaborator

@Si3rr4wow Si3rr4wow commented Feb 12, 2024

Ticket Number

Nav to Appeal pages/ overview
1A - 1D
https://pins-ds.atlassian.net/browse/AAPD-951
https://pins-ds.atlassian.net/browse/AAPD-1074
https://pins-ds.atlassian.net/browse/AAPD-1075
https://pins-ds.atlassian.net/browse/AAPD-1194

Appeal headlines
2A - 2D
https://pins-ds.atlassian.net/browse/AAPD-906
https://pins-ds.atlassian.net/browse/AAPD-1115
https://pins-ds.atlassian.net/browse/AAPD-1116
https://pins-ds.atlassian.net/browse/AAPD-1225

Description of change

This PR adds navigation to the Appeal page/ summary for each party (appellant, LPA, rule 6 party and interested parties).
Note - The nav to rule 6 not yet possible as their dashboard not yet built, but can type in the url to go to a specific appeal.
It also adds the headlines for each page based on the type of user viewing the page.

Checklist

  • Requires infrastructure changes
  • I have updated the documentation accordingly
  • [X ] My commit history in this PR is linear
  • New features have tests
  • Breaking change (team conversation required)

Important

Please do not merge from main (please only rebase). This keeps the history linear and easier to debug.

@Si3rr4wow Si3rr4wow marked this pull request as draft February 12, 2024 10:58
@DeanH1990 DeanH1990 marked this pull request as ready for review February 13, 2024 09:57
Copy link
Contributor

@beejjacobs beejjacobs left a comment

Choose a reason for hiding this comment

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

Few bits to look at, but generally looks good. Note I haven't actually run it to check - but will do.

Copy link
Contributor

@beejjacobs beejjacobs 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!

We will sort the IP business rules separately.

@beejjacobs beejjacobs merged commit b732b9f into main Feb 15, 2024
5 checks passed
@beejjacobs beejjacobs deleted the feature/aapd-951 branch February 15, 2024 08:55
@DeanH1990 DeanH1990 restored the feature/aapd-951 branch February 15, 2024 14:38
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.

4 participants