Skip to content

DTR-5901 Feat-CIS Verify Batch Polling Business Function F7 - #105

Open
abhinavgupta-hmrc wants to merge 8 commits into
mainfrom
DTR-5901
Open

DTR-5901 Feat-CIS Verify Batch Polling Business Function F7#105
abhinavgupta-hmrc wants to merge 8 commits into
mainfrom
DTR-5901

Conversation

@abhinavgupta-hmrc

Copy link
Copy Markdown
Contributor

CIS Verify Batch Polling Business Function F7 - Get Submission with Verification Batch

Comment thread conf/app.routes

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Route style diverges from convention.

All other GET routes in this controller use path parameters (e.g. /cis/verification-batch/newest/:instanceId).
The new endpoint uses query parameters (?instanceId=...&verificationBatchResourceRef=...).

@abhinavgupta-hmrc abhinavgupta-hmrc Jul 13, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Route style updated. Review comment addressed.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Addressed.

@abhinavgupta-hmrc
abhinavgupta-hmrc marked this pull request as ready for review July 13, 2026 07:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants