Skip to content
This repository was archived by the owner on May 29, 2025. It is now read-only.

Conversation

@jmoreira-valory
Copy link
Collaborator

Claim script

@jmoreira-valory jmoreira-valory changed the base branch from develop to main November 28, 2024 09:38
Copy link
Collaborator

@dagacha dagacha left a comment

Choose a reason for hiding this comment

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

Tested - LGTM

Copy link
Collaborator

@Adamantios Adamantios left a comment

Choose a reason for hiding this comment

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

LGTM but lots of constants missing, e.g., safe's url, gas value, gas price, token precision, encoding, etc.

jmoreira-valory and others added 2 commits November 28, 2024 12:40
Co-authored-by: Adamantios Zaras <adamantioszaras@gmail.com>
@jmoreira-valory
Copy link
Collaborator Author

LGTM but lots of constants missing, e.g., safe's url, gas value, gas price, token precision, encoding, etc.

Addressed.

@jmoreira-valory jmoreira-valory merged commit e0dd807 into main Nov 28, 2024
@DavidMinarsch DavidMinarsch deleted the feat/claim_script branch February 5, 2025 15:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants