-
Notifications
You must be signed in to change notification settings - Fork 3
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
Cas3 bed search cleanup #2785
Cas3 bed search cleanup #2785
Conversation
accba19
to
ff4c62e
Compare
...justice/digital/hmpps/approvedpremisesapi/unit/transformer/BedSearchResultTransformerTest.kt
Show resolved
Hide resolved
We need to be careful when migrate this PR because it will break the UI |
@@ -1035,15 +1035,3 @@ components: | |||
- updatedReferenceNumber | |||
- updatedReason | |||
- updatedNotes | |||
ApprovedPremisesBedSearchResult: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this can't be removed currently as it will break the ui compilation and it's non trivial to unpick it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah ok, I did check the UI repo and it only looked like it was being used a a factory, which fred has removed in the PR in the description
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we're now safe to remove this 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please see my comment around ApprovedPremisesBedSearchResult
@muhammad-elabdulla there is a UI pr for that in the PR description, https://github.com/ministryofjustice/hmpps-temporary-accommodation-ui/pull/1191/files |
@daveawc I saw it but if you deploy one of them the UI will break and maybe the ui e2e test not passing. It good to check with @aliuk2012 on Monday |
ff4c62e
to
3cf0c2a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
2e1a270
to
5668b6f
Compare
…it, which will help ongoing work in the area.
5668b6f
to
7e48609
Compare
cleaning up the bed search stuff now that cas1 and cas3 have been split, which will help ongoing work in the area.