Skip to content

Conversation

@rachit77
Copy link
Contributor

@rachit77 rachit77 commented Jan 12, 2026

🔄 Changes Summary

  • Bridge migration file has some conflict compared to release/v0.8 branch
  • 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
@temaniarpit27
Copy link
Contributor

temaniarpit27 commented Jan 12, 2026

@rachit77 pls raise the corresponding PR in release/v0.8
i want to review that first and then this one

@rachit77
Copy link
Contributor Author

@rachit77 pls raise the corresponding PR in release/v0.8 i want to review that first and then this one

#1453

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR resolves migration file conflicts by reorganizing bridge-related schema changes across three migration files. Content that was incorrectly placed in migration 0012 has been moved to a new migration 0014, and a new migration 0013 has been added to set default values for the to_address column.

Changes:

  • Moved table creation statements (backward_let, forward_let, bridge_archive) and source column addition from migration 0012 to new migration 0014
  • Added migration 0013 to set default empty string value for to_address column in bridge table
  • Cleaned up migration 0012 to only contain claim type and index changes plus initial to_address addition

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
bridgesync/migrations/bridgesync0012.sql Removed table creation statements that were moved to migration 0014, keeping only claim type and to_address column additions
bridgesync/migrations/bridgesync0013.sql New migration that sets default value of empty string for to_address column in bridge table
bridgesync/migrations/bridgesync0014.sql New migration containing table creations (backward_let, forward_let, bridge_archive) and source column addition previously in migration 0012

rachit77 added a commit that referenced this pull request Jan 13, 2026
## 🔄 Changes Summary
- Set default value for `to_address` field

## 🔗 Related PRs
- #1452
@sonarqubecloud
Copy link

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