Skip to content

[Tunnel] implement route - Axelar #574

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 25 commits into from
Apr 22, 2025
Merged

[Tunnel] implement route - Axelar #574

merged 25 commits into from
Apr 22, 2025

Conversation

satawatnack
Copy link
Member

@satawatnack satawatnack commented Feb 3, 2025 •
edited
Loading

Implementation details

  • Add Axelar route

Integration test

  • Test send packet through the Axelar route

Please ensure the following requirements are met before submitting a pull request:

  • The pull request is targeted against the correct target branch
  • The pull request is linked to an issue with appropriate discussion and an accepted design OR is linked to a spec that describes the work.
  • The pull request includes a description of the implementation/work done in detail.
  • The pull request includes any and all appropriate unit/integration tests
  • You have added a relevant changelog entry to CHANGELOG_UNRELEASED.md
  • You have re-reviewed the files affected by the pull request (e.g. using the Files changed tab in the Github PR explorer)

@satawatnack satawatnack changed the base branch from master to tunnel-ibc-hook February 3, 2025 08:37
@satawatnack satawatnack requested a review from RogerKSI February 3, 2025 09:09
@RogerKSI RogerKSI requested review from taobun and nkitlabs February 14, 2025 08:48
RogerKSI
RogerKSI requested changes Feb 14, 2025
View reviewed changes
@satawatnack satawatnack requested a review from RogerKSI February 20, 2025 08:55
RogerKSI
RogerKSI reviewed Feb 20, 2025
View reviewed changes
@satawatnack satawatnack requested a review from RogerKSI February 21, 2025 08:25
@RogerKSI RogerKSI requested a review from colmazia March 23, 2025 15:31
nkitlabs
nkitlabs reviewed Apr 1, 2025
View reviewed changes
Copy link
Contributor

@nkitlabs nkitlabs left a comment

Choose a reason for hiding this comment

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

branch is conflicted, please help fix

Base automatically changed from tunnel-ibc-hook to master April 2, 2025 15:24
@satawatnack satawatnack requested a review from nkitlabs April 3, 2025 09:12
@satawatnack satawatnack requested a review from RogerKSI April 9, 2025 06:00
nkitlabs
nkitlabs previously approved these changes Apr 10, 2025
View reviewed changes
RogerKSI
RogerKSI reviewed Apr 10, 2025
View reviewed changes
@satawatnack satawatnack requested a review from RogerKSI April 10, 2025 06:46
RogerKSI
RogerKSI previously approved these changes Apr 10, 2025
View reviewed changes
Base automatically changed from tunnel-router-route to master April 11, 2025 04:09
@RogerKSI RogerKSI dismissed stale reviews from nkitlabs and themself April 11, 2025 04:09

The base branch was changed.

@RogerKSI RogerKSI requested review from RogerKSI and nkitlabs April 16, 2025 07:09
RogerKSI
RogerKSI previously approved these changes Apr 16, 2025
View reviewed changes
@RogerKSI RogerKSI marked this pull request as draft April 16, 2025 07:09
@RogerKSI RogerKSI marked this pull request as ready for review April 16, 2025 07:10
nkitlabs
nkitlabs previously approved these changes Apr 16, 2025
View reviewed changes
@satawatnack satawatnack dismissed stale reviews from nkitlabs and RogerKSI via 2e98a83 April 18, 2025 09:40
RogerKSI
RogerKSI previously approved these changes Apr 18, 2025
View reviewed changes
@RogerKSI RogerKSI requested a review from nkitlabs April 18, 2025 10:24
taobun
taobun reviewed Apr 21, 2025
View reviewed changes
@satawatnack satawatnack requested a review from taobun April 21, 2025 10:47
@RogerKSI RogerKSI merged commit 7e99c72 into master Apr 22, 2025
2 checks passed
@RogerKSI RogerKSI deleted the tunnel-axelar-route branch April 22, 2025 06:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@RogerKSI RogerKSI RogerKSI approved these changes

@taobun taobun taobun approved these changes

@colmazia colmazia Awaiting requested review from colmazia

@nkitlabs nkitlabs Awaiting requested review from nkitlabs

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@satawatnack @RogerKSI @CodiumAI-Agent @taobun @nkitlabs