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

Hotfix/jm 8074 #735

Merged
merged 5 commits into from
Aug 23, 2024
Merged

Hotfix/jm 8074 #735

merged 5 commits into from
Aug 23, 2024

Conversation

akikrahman1
Copy link
Contributor

JIRA link (if applicable)

https://centralgovernmentcgi.atlassian.net/browse/JM-8074

Change description

Update to avoid error when changing deferrals on same day where a letter could already be queued

Does this PR introduce a breaking change? (check one with "x")

[ ] Yes
[ x] No

@akikrahman1 akikrahman1 requested a review from a team as a code owner August 22, 2024 17:50
@akikrahman1 akikrahman1 changed the base branch from master to Stabilisation13 August 23, 2024 10:50
@akikrahman1 akikrahman1 merged commit ca6a8f6 into Stabilisation13 Aug 23, 2024
7 checks passed
@akikrahman1 akikrahman1 deleted the hotfix/JM-8074 branch August 23, 2024 11:50
Ben-Edwards-cgi added a commit that referenced this pull request Aug 28, 2024
* Updated Ballot reports to be ordered by juror last name (#730)

* update daily-utilisation-jurors to not fill up a null column if there is no entries for that specific day (#731)

* Fixed issue where attendance and payment history would show the incorrect data (#733)

* Hotfix/jm 8074 (#735)

* hotfix/JM-8074 Technical error when changing the deferral date using the "Change" button on the overview page

* reverting unwanted update for flyway

* Update build.gradle

* Update build.gradle

* updates for PMD and checkstyles

* Update README.md (#574)

* Update README.md

* Update README.md

* Update CreatePoolControllerITest.java

* Use Lazy loading instead of Eager to reduce database calls (#740)

* Updated Many classes to use Lazy Loading

* Removed comment

* Applied review comments

* Updated the way we show the jurors location to only show if they are checked in or on a trial (#745)

* Filtering on service start date on active pool screen only applies to page you filter on (#732)

* Updated pools-requested endpoint to use pagination Utils

* Fixed styles

* Updated active pools to use pagination utils

* Applied review comments

* update filters to add missing and update wrong ones (#743)

* update filters to add missing and update wrong ones

* update how we group-by on active pools filtering

* Fixed styles

---------

Co-authored-by: Ricardo <140706972+ricardofreitasrocha@users.noreply.github.com>

* Added includePanelMembers to person attending summary and detailed reports (#746)

* Fixed tests

* Remove JurorPoolUtils.getSingleActiveJurorPool() and replace with jurorPoolSerive.getActiveRecordFromUser() (#744)

* Updated the way we retrieve juror pools to prevent non-unique entry errors

* Applied review comments

* Fixed styles / merged stabilisation13

* fix_random_fail_test (#747)

* Permission change to allow the Bureau general users to edit change name / fix error in name option (#748)

* Updated permissions needed to fix error in current name

* Applied review comments

* hotfix/JM-8103 update to utilisation report functions (#749)

* hotfix/JM-8103 update to utilisation report functions

* updating tests

* updates following business review

* Added comments to sql file

* Active Pools Not Displaying Records (#750)

* Removed limits for active pool and pool requests

* Fixed styles

* Fixed styles

* Fixed issue where attendance audit number would incorrectly be added

* Fixed issue where attendance audit number would incorrectly be added

* Fixed styles

---------

Co-authored-by: Ricardo <140706972+ricardofreitasrocha@users.noreply.github.com>
Co-authored-by: akikrahman1 <149579836+akikrahman1@users.noreply.github.com>
Co-authored-by: akikrahman1 <akhlaqur.rahman1@hmcts.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants