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

[Property] OriginalCurrency field should not default to 0 #187

Open
3 of 4 tasks
johcarter opened this issue Jun 5, 2024 · 1 comment
Open
3 of 4 tasks

[Property] OriginalCurrency field should not default to 0 #187

johcarter opened this issue Jun 5, 2024 · 1 comment
Assignees
Labels
minor minor release (non-breaking change)

Comments

@johcarter
Copy link
Contributor

johcarter commented Jun 5, 2024

Description

change the default value of OriginalCurrency to n/a from 0 for consistency with other currency fields.

Reasons for change

All values in a currency field must be datatype char(3)

Scope of change

  • Location File
  • Accounts File
  • Reinsurance Scope
  • Reinsurance Info

Impact of change

minor/non-breaking

Data type, default values, list valid values

Field Name Data type Default values Valid values
OriginalCurrency char(3) n/a
@johcarter johcarter added the minor minor release (non-breaking change) label Jun 5, 2024
@johcarter johcarter self-assigned this Jun 5, 2024
johcarter added a commit that referenced this issue Jun 26, 2024
@MattDonovan82
Copy link
Contributor

MattDonovan82 commented Jul 12, 2024

no further comments. moving to done.

sambles pushed a commit that referenced this issue Jul 17, 2024
sambles pushed a commit that referenced this issue Jul 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor minor release (non-breaking change)
Projects
Status: Done
Status: Done
Status: Agreed and ready
Development

No branches or pull requests

2 participants