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

As an LA I can confirm all the scheme details are up-to-date #64

Closed
markhobson opened this issue Jan 10, 2024 · 1 comment
Closed

As an LA I can confirm all the scheme details are up-to-date #64

markhobson opened this issue Jan 10, 2024 · 1 comment
Assignees
Labels
5sp 5 story points
Milestone

Comments

@markhobson
Copy link
Contributor

markhobson commented Jan 10, 2024

As an LA I can confirm all the scheme details are up-to-date so that I don't have to make unnecessary changes.

Acceptance criteria

  1. If a scheme needs review, then the scheme page highlights this above the fold
  2. A review question is added to the end of the scheme page
  3. This question asks if the scheme is up-to-date
  4. If the user checks a box to confirm that the details are up-to-date, then the "last reviewed" timestamp is updated
@Sparrow0hawk Sparrow0hawk added this to the 1.0.0 milestone Jan 24, 2024
@Sparrow0hawk Sparrow0hawk modified the milestones: 1.0.0, 0.11.0 Feb 7, 2024
@Sparrow0hawk Sparrow0hawk added the 5sp 5 story points label Feb 13, 2024
@Sparrow0hawk Sparrow0hawk modified the milestones: 0.11.0, 0.12.0 Feb 21, 2024
@markhobson markhobson changed the title As an LA I can confirm all the scheme details are up to date As an LA I can confirm all the scheme details are up-to-date Feb 28, 2024
markhobson added a commit that referenced this issue Feb 29, 2024
@markhobson
Copy link
Contributor Author

markhobson commented Mar 4, 2024

Tasks

  • Allow scheme to be reviewed
  • Add CSRF token
  • Endpoint needs to check checkbox
  • Endpoint needs to add authority review
  • Add error message for not checking the box
  • Add success notification after review
  • Check different authority
  • Pull up update to domain? Introduce reviews domain object?
  • Move review stuff into review module (view)
  • Move needs review tag into inset text
  • Move last reviewed out of overview down into review question

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5sp 5 story points
Projects
None yet
Development

No branches or pull requests

2 participants