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

RIA-8802 bails notice of hearing displays the correct location information #880

Conversation

DennisHoV1
Copy link
Contributor

Jira link (if applicable)

https://tools.hmcts.net/jira/browse/RIA-8802

Change description

  • display the listing location from ref data instead of using CCD address
  • update unit test

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

Copy link
Contributor

@augustio augustio left a comment

Choose a reason for hiding this comment

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

LGTM.

@DennisHoV1 DennisHoV1 merged commit 709d744 into ia-case-documents-api-refdata Apr 24, 2024
5 checks passed
@DennisHoV1 DennisHoV1 deleted the RIA-8802-bails-notice-of-hearing-displays-the-correct-location-information branch April 24, 2024 09:30
rajeshthuraiyur added a commit that referenced this pull request Aug 15, 2024
* RIA-8802 bails notice of hearing displays the correct location information (#880)

* RIA-8802 bails notice of hearing displays the correct location information

* update the imported library and indentation

* RIA-9437 - Fix for missing listCaseHearingCentre error when submitting Prepare decision and reasons event (#943)

* RIA-9437 - Fix for missing listCaseHearingCentre error when submitting Prepare decision and reasons event

* FIx checkstyle

---------

Co-authored-by: DennisHoV1 <106669367+DennisHoV1@users.noreply.github.com>
Co-authored-by: rajeshthuraiyur <48379561+rajeshthuraiyur@users.noreply.github.com>
ThomasKKC added a commit that referenced this pull request Aug 22, 2024
* DIAC-555 add logs (#934)

* Added logs

* Added logs

* DIAC-556 Update OWASP and gradle (#935)

* Update owasp dependency check

* Update owasp dependency check

* Update owasp dependency check

* Update gradle

* Update sonarqube

---------

Co-authored-by: jamiephillipshmcts <123951690+jamiephillipshmcts@users.noreply.github.com>

* DIAC-556 Revert "DIAC-556 Update OWASP and gradle" (#938)

This reverts commit 35756b8.

* DIAC-556 Remove JCenter (#941)

* Update owasp dependency check

* Update owasp dependency check

* Update owasp dependency check

* Update gradle

* Update sonarqube

* remove jcenter()

* add logstash

* remove jcenter in buildscript

* RIA-TASK - DO NOT MERGE - REF DATA (#944)

* RIA-8802 bails notice of hearing displays the correct location information (#880)

* RIA-8802 bails notice of hearing displays the correct location information

* update the imported library and indentation

* RIA-9437 - Fix for missing listCaseHearingCentre error when submitting Prepare decision and reasons event (#943)

* RIA-9437 - Fix for missing listCaseHearingCentre error when submitting Prepare decision and reasons event

* FIx checkstyle

---------

Co-authored-by: DennisHoV1 <106669367+DennisHoV1@users.noreply.github.com>
Co-authored-by: rajeshthuraiyur <48379561+rajeshthuraiyur@users.noreply.github.com>

* Fix rebase

---------

Co-authored-by: apereverzin1 <140609226+apereverzin1@users.noreply.github.com>
Co-authored-by: Amanda Richards <47543618+AmandaRichards@users.noreply.github.com>
Co-authored-by: jamiephillipshmcts <123951690+jamiephillipshmcts@users.noreply.github.com>
Co-authored-by: Nilay <nilay.patel@solirius.com>
Co-authored-by: robin-ieong <101253854+robin-ieong@users.noreply.github.com>
Co-authored-by: DennisHoV1 <106669367+DennisHoV1@users.noreply.github.com>
Co-authored-by: rajeshthuraiyur <48379561+rajeshthuraiyur@users.noreply.github.com>
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