Skip to content

Conversation

@rachit77
Copy link
Contributor

🔄 Changes Summary

  • Set default value for to_address field

🔗 Related PRs

@rachit77 rachit77 self-assigned this Jan 12, 2026
@rachit77 rachit77 mentioned this pull request Jan 12, 2026
@rachit77 rachit77 marked this pull request as ready for review January 12, 2026 13:26
@sonarqubecloud
Copy link

Copy link
Contributor

@goran-ethernal goran-ethernal left a comment

Choose a reason for hiding this comment

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

I think it is good enough. Although, on migrate down, I would still add the update query to update this column to an empty string just in case if someone want's to run that version, even though the chances for that are small.

@rachit77
Copy link
Contributor Author

I think it is good enough. Although, on migrate down, I would still add the update query to update this column to an empty string just in case if someone want's to run that version, even though the chances for that are small.

this previous version of to_address is used by nobody, this feature will only go live with v0.8 release

@rachit77 rachit77 merged commit fd98fb7 into release/0.8 Jan 13, 2026
22 checks passed
@rachit77 rachit77 deleted the v0.8-fix-migration branch January 13, 2026 08:43
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.

5 participants