Skip to content
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

uLxLy Bridge Commands #448

Merged
merged 25 commits into from
Jan 9, 2025
Merged

uLxLy Bridge Commands #448

merged 25 commits into from
Jan 9, 2025

Conversation

praetoriansentry
Copy link
Member

@praetoriansentry praetoriansentry commented Dec 5, 2024 •
edited
Loading

Description

This PR adds a bunch of commands to polycli that can be used for making and claiming bridge deposits. Claims will depend on the bridge service.

@praetoriansentry
feat: copying for alonso
adb8e80
@praetoriansentry
refactor: switching to cobra.. needs testing
3f9f59e
@praetoriansentry
fix: minor issues related to missing field and 0x prefix
f0e026d
@praetoriansentry
fix: deposit_cnt type, missing function call, call data in wrong level
0bf4f49
@praetoriansentry
feat: adding ability to override global index while the bug is fixed
9a6b9eb
@praetoriansentry
fix: dropping string tag
a993921
@praetoriansentry
feat: adding dry-run and gas-price
5034916
@praetoriansentry
feat: better logging of revert data
9006c1a
@praetoriansentry
fix: issue with bridge address
cd1a382
@praetoriansentry
fix: token too long for large bridge messages
d4875fc
@praetoriansentry
fix: handling 0x prefix encoding issue
7319e96
@praetoriansentry
feat: intial implementation for claim-everything
7a3919b
@praetoriansentry
feat: adding more logging
b88ca09
@praetoriansentry
feat: additional logging
7de0bfb
@praetoriansentry
feat: surpressing error logs
cfd970c
@praetoriansentry
fix: null tx
971e19f
@praetoriansentry
feat: better defaults for dry running
7fa6ed2
@praetoriansentry
fix: dropping flag that should not be needed
b4ac5bc
@praetoriansentry
fix: some lint issues
9694725
@praetoriansentry
fix: generating docs
410b2f1
@praetoriansentry
Merge branch 'main' into jhilliard/alonso
0cfb130
@praetoriansentry
docs: updating some of the examples and docs
0f4cc09
leovct
leovct approved these changes Jan 7, 2025
View reviewed changes
Copy link
Member

@leovct leovct left a comment

Choose a reason for hiding this comment

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

lgtm! I took a quick look at the code and suggested some very minor changes to the docs :)

cmd/ulxly/delete.md Outdated Show resolved Hide resolved
cmd/ulxly/ulxly.go Outdated Show resolved Hide resolved
cmd/ulxly/ulxly.go Outdated Show resolved Hide resolved
cmd/ulxly/ulxly.go Outdated Show resolved Hide resolved
cmd/ulxly/ulxly.go Outdated Show resolved Hide resolved
cmd/ulxly/ulxly.go Show resolved Hide resolved
cmd/ulxly/ulxly.go Show resolved Hide resolved
cmd/ulxly/ulxly.go Outdated Show resolved Hide resolved
cmd/ulxly/ulxly.go Outdated Show resolved Hide resolved
cmd/ulxly/ulxly.go Outdated Show resolved Hide resolved
@praetoriansentry praetoriansentry marked this pull request as ready for review January 9, 2025 12:36
@praetoriansentry praetoriansentry changed the title ulxly commands uLxLy Bridge Commands Jan 9, 2025
@praetoriansentry praetoriansentry merged commit e6526b6 into main Jan 9, 2025
6 checks passed
@praetoriansentry praetoriansentry deleted the jhilliard/alonso branch January 9, 2025 12:46
@leovct leovct mentioned this pull request Jan 9, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@leovct leovct leovct approved these changes

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

Successfully merging this pull request may close these issues.
2 participants
@praetoriansentry @leovct