Skip to content

CDRW-4775 codeset repo cleanup#6

Merged
cjrobbertse-ob merged 19 commits intov4.0.1-draft-1from
CDRW-4775-Update-the-OB-Codeset-repo-with-below-changes
Nov 12, 2025
Merged

CDRW-4775 codeset repo cleanup#6
cjrobbertse-ob merged 19 commits intov4.0.1-draft-1from
CDRW-4775-Update-the-OB-Codeset-repo-with-below-changes

Conversation

@cjrobbertse-ob
Copy link
Contributor

@cjrobbertse-ob cjrobbertse-ob commented Nov 11, 2025

added a mixture of Tanu's changes and my own for clean up. See Jira Ticket for more info

https://openbanking.atlassian.net/browse/CDRW-4775

Ticket description:

  1. Remove extra spaces from CodeName and move descriptive text from Code Name to 'Description' column where applicable
  2. Remove the extra header from OB_Internal_Codeset.

@cjrobbertse-ob cjrobbertse-ob marked this pull request as ready for review November 11, 2025 12:44
@cjrobbertse-ob
Copy link
Contributor Author

@OBPeteS please confirm you are happy with points 1 and 2 from the ticket.

Please also check you are happy with bulk changes from Tanu. (Very hard to track changes, but I did go through by hand comparing what was needed and not...)

Copy link
Contributor

@OBPeteS OBPeteS left a comment

Choose a reason for hiding this comment

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

Suggested a fix for the blue text not rendering as blue, please test and see if it works

@cjrobbertse-ob cjrobbertse-ob changed the title codeset repo cleanup CDRW-4775 codeset repo cleanup Nov 12, 2025
Copy link
Contributor

@OBPeteS OBPeteS left a comment

Choose a reason for hiding this comment

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

Looks good

@cjrobbertse-ob cjrobbertse-ob merged commit ab74336 into v4.0.1-draft-1 Nov 12, 2025
@cjrobbertse-ob cjrobbertse-ob deleted the CDRW-4775-Update-the-OB-Codeset-repo-with-below-changes branch November 12, 2025 16:32
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