fix(applicaiton-system): Payment migration for definition to text from string #16529
+27
−0
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Update
definition
Column Type to TEXT in Payment TableDescription
This PR introduces a migration to update the
definition
column in thepayment
table fromSTRING
toTEXT
. This change is necessary to accommodate larger JSON schema data that exceeds the previous character limit imposed by theSTRING
type.What
definition
column type fromSequelize.STRING
toSequelize.TEXT
in thepayment
table.up
method to change the column type toTEXT
.down
method to revert the column type back toSTRING
if needed.Why
TEXT
type allows for storing larger strings, which is essential for handling JSON data that can exceed the defaultSTRING
length limit.Testing
Notes
TEXT
can store all data types previously stored inSTRING
.TEXT
andSTRING
similarly in Node.js.Screenshots / Gifs
Attach Screenshots / Gifs to help reviewers understand the scope of the pull request
Checklist: