-
Notifications
You must be signed in to change notification settings - Fork 19
feat: integrate RNS support across transfer, batch-transfer, and balance commands #205
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
Open
97woo
wants to merge
21
commits into
rsksmart:main
Choose a base branch
from
97woo:feat/add-rns-resolve-command
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.
Open
Changes from all commits
Commits
Show all changes
21 commits
Select commit
Hold shift + click to select a range
1b32861
feat: add RNS resolve command
97woo ed4aff6
Merge upstream/main and resolve conflicts
97woo c64d1d8
docs: add RNS resolve command documentation to README
97woo 3ac897a
feat: add RNS helper module for domain resolution
97woo 5721792
feat: add RNS support to transfer command
97woo d509f44
feat: add RNS support to batch-transfer command
97woo 75261b3
docs: update README with RNS integration examples
97woo 1be9f1e
docs: use verified RNS domains in examples
97woo e5b0ad4
fix: resolve merge conflicts with upstream/main
97woo 1925b56
refactor: update RNS implementation based on PR review
97woo 6194850
style: apply wallet.ts pattern with helper functions
97woo 002e058
refactor: apply object parameter pattern to RNS functions
97woo dfdb7e2
fix: resolve RNS module import issues and remove code comments
97woo 987a1ec
fix: improve code quality and type safety
97woo eb69e44
fix: address PR review comments for RNS integration
97woo f05b834
Merge remote-tracking branch 'upstream/main' into feat/add-rns-resolv…
97woo 7c79ead
fix: final cleanup - fix indentation and remove comment
97woo 1018969
fix: improve RNS domain validation with proper length check
97woo aa44fc8
feat(utils): add Rootstock EIP-1191/EIP-55 address helpers
97woo db1e42d
fix(rns/resolve/cli): apply chain-aware validation and normalization
97woo 58d757c
fix(balance): use provided address first; fallback to rBTC if prompt …
97woo 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 hidden or 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 hidden or 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.
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.
we shoould call it just resolve?
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.
Yes it's just resolve