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

Cruikshanks latest changes to the view return log #1621

Merged

Conversation

Cruikshanks
Copy link
Member

@Cruikshanks Cruikshanks commented Jan 15, 2025

https://eaflood.atlassian.net/browse/WATER-4745

See Create Completed Monthly Return Logs page

The following changes are included

  • switch to existing status macro and remove duplicate
  • set correct status in presenter
  • update versions design (add 'Version x' as a line and prefix with 'Viewing ' for selected version)
  • add received date to the metadata
  • update all return log links to go to view page
  • update Edit return to be Edit or Submit return (depending on return log status)
  • update page to handle viewing any return log, including 'due' and 'received' which have no submission data

@Cruikshanks Cruikshanks added the do not merge Used for spikes and experiments label Jan 15, 2025
@Cruikshanks Cruikshanks self-assigned this Jan 15, 2025
@Cruikshanks Cruikshanks force-pushed the alans-latest-update branch 4 times, most recently from 153272b to 817aefa Compare January 15, 2025 23:07
https://eaflood.atlassian.net/browse/WATER-4745

> See [Create Completed Monthly Return Logs page](#1484)

> Will not work until `main` is merged into `new-view-completed-monthly-return-logs-page`

The following changes are included

- switch to existing status macro and remove duplicate
- set correct status in presenter
- update versions design (add 'Version x' as a line and prefix with 'Viewing ' for selected version)
- add received date to the metadata
- update all return log links to go to view page
- update Edit return to be Edit or Submit return (depending on return log status)
- update page to handle viewing _any_ return log, including 'due' and 'received' which have no submission data
@StuAA78 StuAA78 marked this pull request as ready for review January 16, 2025 16:22
@StuAA78 StuAA78 merged commit 80deedb into new-view-completed-monthly-return-logs-page Jan 16, 2025
2 of 3 checks passed
@StuAA78 StuAA78 deleted the alans-latest-update branch January 16, 2025 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge Used for spikes and experiments
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants