Skip to content

Refactor Api::V1::FormResource #make_live! and #archive!#2071

Merged
lfdebrux merged 1 commit into
mainfrom
ldeb-refactor-form-resource-make-live-archive
Jul 24, 2025
Merged

Refactor Api::V1::FormResource #make_live! and #archive!#2071
lfdebrux merged 1 commit into
mainfrom
ldeb-refactor-form-resource-make-live-archive

Conversation

@lfdebrux

Copy link
Copy Markdown
Contributor

What problem does this pull request solve?

Change form resource make_live! and archive! methods to update the form object in-place with the response from the API.

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?
  • Do the end to end tests need updating before these changes will pass?
  • Has all relevant documentation been updated?

@lfdebrux lfdebrux force-pushed the ldeb-refactor-form-resource-make-live-archive branch from 9ef4acb to 4fe579d Compare July 11, 2025 12:20
@lfdebrux lfdebrux changed the title Refactor Api::V1::FormResource #make_live! and #archive! Refactor Api::V1::FormResource #make_live! and #archive! Jul 15, 2025
@lfdebrux lfdebrux force-pushed the ldeb-refactor-form-resource-make-live-archive branch from 4fe579d to f62ca39 Compare July 18, 2025 07:47
@github-actions

Copy link
Copy Markdown

🎉 A review copy of this PR has been deployed! You can reach it at: https://pr-2071.admin.review.forms.service.gov.uk/

It may take 5 minutes or so for the application to be fully deployed and working. If it still isn't ready
after 5 minutes, there may be something wrong with the ECS task. You will need to go to the integration AWS account
to debug, or otherwise ask an infrastructure person.

For the sign in details and more information, see the review apps wiki page.

@lfdebrux lfdebrux enabled auto-merge July 24, 2025 09:36
Change form resource make_live! and archive! methods to update the form
object in-place with the response from the API.
@lfdebrux lfdebrux force-pushed the ldeb-refactor-form-resource-make-live-archive branch from f62ca39 to b2182e3 Compare July 24, 2025 09:36
@sonarqubecloud

Copy link
Copy Markdown

@lfdebrux lfdebrux merged commit 629c3bf into main Jul 24, 2025
4 of 5 checks passed
@lfdebrux lfdebrux deleted the ldeb-refactor-form-resource-make-live-archive branch July 24, 2025 09:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants