Skip to content

CARF-435 Change Link- Do you have a Unique Taxpayer Reference?#152

Merged
niranchanaa-seshadri merged 6 commits into
mainfrom
CARF-435-clean
May 29, 2026
Merged

CARF-435 Change Link- Do you have a Unique Taxpayer Reference?#152
niranchanaa-seshadri merged 6 commits into
mainfrom
CARF-435-clean

Conversation

@simondrugan16
Copy link
Copy Markdown
Contributor

Ticket: CARF-435

Test by going through the journey with scenarios in table on ticket, changing the value, and making sure the user gets to the correct url and the correct answers get changed

Comment thread test/base/SpecBase.scala
writes: Writes[T],
rds: Reads[T]
): UserAnswers = {
val updatedData = userAnswers.data.setObject(page.path, Json.toJson(value)) match {
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.

Does this fix my bug I found for tests? (however, still valid for main code)

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.

It fixes it for tests, yeah. The cleanup methods only effect things downstream of them, and only clears things if it's no longer valid, so it's more of a feature than a bug. (still need to be careful with it though)

ZDevelop94
ZDevelop94 previously approved these changes May 27, 2026
@niranchanaa-seshadri niranchanaa-seshadri merged commit f3d588e into main May 29, 2026
1 check passed
@niranchanaa-seshadri niranchanaa-seshadri deleted the CARF-435-clean branch May 29, 2026 08:30
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.

3 participants