Skip to content

Comments

Render separate view for making draft of archived form live#1067

Merged
stephencdaly merged 1 commit intomainfrom
1433-render-make-archive-live-page-for-making-archive-draft-live
Apr 3, 2024
Merged

Render separate view for making draft of archived form live#1067
stephencdaly merged 1 commit intomainfrom
1433-render-make-archive-live-page-for-making-archive-draft-live

Conversation

@stephencdaly
Copy link
Contributor

@stephencdaly stephencdaly commented Apr 3, 2024

What problem does this pull request solve?

Trello card: https://trello.com/c/UVCBeTs8/1433-allow-user-to-create-a-draft-for-an-archived-form

We will have different content for the confirm page for making the draft of an archived form live. Render a separate view for this - make_archived_draft_live.html.erb. This view currently duplicates the content for making the draft of a live form live, and the content will need to be updated when it is ready.

This will render the same content regardless of whether the form is in archived or archived_with_draft state. The form will only transition to archived_with_draft state if edits are made before making the draft live. We have a separate route and controller for making the form live again without making a draft, so the content for that is separate.

Things to consider when reviewing

  • Ensure that you consider the wider context.
  • Does it work when run on your machine?
  • Is it clear what the code is doing?
  • Do the commit messages explain why the changes were made?
  • Are there all the unit tests needed?
  • Has all relevant documentation been updated?

We will have different content for the confirm page for making the
draft of an archived form live. Render a separate view for this -
`make_archived_draft_live.html.erb`. This view currently duplicates
the content for making the draft of a live form live, and the content
will need to be updated when it is ready.

This will render the same content regardless of whether the form is in
`archived` or `archived_with_draft` state. The form will only
transition to `archived_with_draft` state if edits are made before
making the draft live. We have a separate route and controller for
making the form live again without making a draft, so the content for
that is separate.
@stephencdaly stephencdaly force-pushed the 1433-render-make-archive-live-page-for-making-archive-draft-live branch from f06df40 to b585d8d Compare April 3, 2024 14:57
@sonarqubecloud
Copy link

sonarqubecloud bot commented Apr 3, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@stephencdaly stephencdaly marked this pull request as ready for review April 3, 2024 15:15
@stephencdaly stephencdaly merged commit 0a82064 into main Apr 3, 2024
@stephencdaly stephencdaly deleted the 1433-render-make-archive-live-page-for-making-archive-draft-live branch April 3, 2024 15:16
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