-
Notifications
You must be signed in to change notification settings - Fork 5
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
chalabi/cross chain x #194
Open
chalabi2
wants to merge
45
commits into
main
Choose a base branch
from
chalabi/cross-chain-x
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+1,559
−323
Open
Changes from all commits
Commits
Show all changes
45 commits
Select commit
Hold shift + click to select a range
2c7c09d
chore: remove hardcoded chainNames
chalabi2 a7aceac
feat: add osmosis balances
chalabi2 2d1826e
chore: add ibc mocks, token filters/queries, addresses etc
chalabi2 178dd55
successful ibc tx generation
chalabi2 27b7338
fix: timeout time/height in ibcTransfer
chalabi2 0020ea8
Merge branch 'main' into chalabi/cross-chain-x
chalabi2 758500e
feat: add switcher to ibc chain selectors
chalabi2 2a4826a
chore: add group ibc functionality
chalabi2 03619b3
fix: show ibc assets in history
chalabi2 0ec0be6
Merge branch 'main' into chalabi/cross-chain-x
chalabi2 dc163e0
chore: add back uppercase
chalabi2 aa9301f
chore: code rabbit suggestions
chalabi2 6f565da
chore: code rabbit suggestions
chalabi2 dbeeec5
fix: merge conflicts
chalabi2 5a0f4c1
chore: add osmosis related .env variables to readme
chalabi2 f7ce672
fix: re-add proper rendering logic to bank page
chalabi2 5cea9f3
Update README.md
chalabi2 fb9cfff
Update README.md
chalabi2 09bf705
Update config/env.ts
chalabi2 7b1a642
chore: add coderabbit changes
chalabi2 6d570b0
Update config/env.ts
chalabi2 bd0319f
Update config/env.ts
chalabi2 958a27f
Update components/bank/components/sendBox.tsx
chalabi2 de95589
Update pages/_app.tsx
chalabi2 1e474cf
Update pages/bank.tsx
chalabi2 036be3f
Update scripts/ibcTransferAll.ts
chalabi2 65dbb12
Update scripts/ibcTransferAll.ts
chalabi2 f0e7e8d
Update components/bank/forms/ibcSendForm.tsx
chalabi2 52672ce
Update components/bank/forms/ibcSendForm.tsx
chalabi2 d7bcb83
Update components/bank/forms/ibcSendForm.tsx
chalabi2 3e1c22f
Update components/bank/forms/ibcSendForm.tsx
chalabi2 38129b1
Update components/bank/forms/ibcSendForm.tsx
chalabi2 16da549
Update components/bank/forms/ibcSendForm.tsx
chalabi2 ed1b3ee
Update components/bank/forms/ibcSendForm.tsx
chalabi2 c748bc0
Update hooks/useLcdQueryClient.ts
chalabi2 3ff70b4
Update hooks/useLcdQueryClient.ts
chalabi2 e50afae
Update hooks/useTx.tsx
chalabi2 891c4b6
chore: add transfer script instructions
chalabi2 2cd2f92
chore: prettier
chalabi2 5fef3a4
fix: fix form tests
chalabi2 6562154
fix: add mocked osmosis asset list
chalabi2 915a268
fix: move chain selection test to ibc test file
chalabi2 d6d8ddb
fix: incrase wait time for slow ci machine
chalabi2 e367ae2
remove loading state from ibc form mock in sendbox
chalabi2 a47b3d2
fix: proper module mock path for ibcSendForm in sendbox test
chalabi2 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -38,4 +38,6 @@ next-env.d.ts | |
|
||
.idea/ | ||
|
||
certificates | ||
certificates | ||
|
||
/scripts/demons.json |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🛠️ Refactor suggestion
Extract IBC metadata handling and add safety checks.
The code has two issues:
assetInfo
propertiesExtract the logic into a utility function:
📝 Committable suggestion