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.

Sign up for GitHub

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

Jump to bottom

Dfr 3543 ct judiciary upload multiple attachments function #2109

Open
wants to merge 40 commits into
base: master
Choose a base branch
from
Open

Dfr 3543 ct judiciary upload multiple attachments function #2109

wants to merge 40 commits into from

Conversation

hamzahtahirhmcts
Copy link
Contributor

Jira link

DFR-3543

See PROJ-XXXXXX

Change description

Testing done

Checklist

  • commit messages are meaningful and follow good commit message guidelines
  • README and other documentation has been updated / added (if needed)
  • tests have been updated / new tests has been added (if needed)
  • Does this PR introduce a breaking change

jthmcts
jthmcts reviewed Jan 30, 2025
View reviewed changes
Copy link
Contributor

@jthmcts jthmcts left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can also remove the code from HearingOrderController.storeHearingOrder as this will no longer be used

dawudgovuk
dawudgovuk previously approved these changes Jan 30, 2025
View reviewed changes
@hamzahtahirhmcts hamzahtahirhmcts dismissed stale reviews from dawudgovuk and al-hmcts via 78c8329 January 30, 2025 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ptrelease ptrelease ptrelease approved these changes

@so99y so99y so99y approved these changes

@jthmcts jthmcts jthmcts approved these changes

@dawudgovuk dawudgovuk dawudgovuk left review comments

@al-hmcts al-hmcts al-hmcts left review comments

Assignees
No one assigned
Labels
ns:financial-remedy prd:finrem rel:finrem-cos-pr-2109
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@hamzahtahirhmcts @ptrelease @so99y @jthmcts @dawudgovuk @al-hmcts