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.

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

Python client & tests improvements #498

Merged
merged 14 commits into from
Dec 1, 2023

Conversation

apaillier-ledger
Copy link
Contributor

Description

Now relies on the Web3.py library for tests.
Greatly simplifies both the tests client as well as the tests themselves.

Changes include

  • Bugfix (non-breaking change that solves an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (change that is not backwards-compatible and/or changes current functionality)
  • Tests
  • Documentation
  • Other (for changes that might not fit in any category)

@apaillier-ledger apaillier-ledger force-pushed the feat/apa/python_client_improvements branch 2 times, most recently from 1a57079 to 0478854 Compare November 21, 2023 10:31
@apaillier-ledger apaillier-ledger force-pushed the feat/apa/python_client_improvements branch 2 times, most recently from 04b3cd2 to 816a179 Compare November 29, 2023 17:04
- recover_message / recover_transaction
- get_selector_from_data
Now uses the ERC 721/1155 ABIs, gets selector automatically, and actually verifies the signature
@apaillier-ledger apaillier-ledger force-pushed the feat/apa/python_client_improvements branch from 816a179 to f5120ab Compare November 29, 2023 17:25
@apaillier-ledger apaillier-ledger force-pushed the feat/apa/python_client_improvements branch from f5120ab to 67a472d Compare November 30, 2023 09:57
@apaillier-ledger apaillier-ledger merged commit 4bf26a7 into develop Dec 1, 2023
80 of 90 checks passed
@apaillier-ledger apaillier-ledger deleted the feat/apa/python_client_improvements branch December 1, 2023 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants