gitcoinco / gitcoinco/web

upgrade web3.py

Open
#9,757 3 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
1.8k
Forks
776
PR merge metrics
No merged PRs in 30d

Description

### Description

In order to take advantage of the London fork, we need to upgrade web3.py to the latest version (5.22.0)

Some work on migrating v4 -> v5 has already been undertaken here (by developerfred): https://github.com/gitcoinco/web/pull/6366

However, developerfred was working to upgrade web3.py to 5.7.0 (back in April of 2020) and it's likely that many things (especially with regards to the London fork) will not have been implemented.

The team believes it would be better to start a fresh branch and to make all of the required changes again using #6366 as a guide (along with the migration guide found here).

Upgrades will need to be made against any script that utilises web3.py:

- /app/avatar/management/commands/create_text_inputs.py
- /app/dashboard/models.py
- /app/dashboard/poh_utils.py
- /app/dashboard/tip_views.py
- /app/dashboard/utils.py
- /app/dashboard/views.py
- /app/dashboard/management/commands/send_bulk_tips.py
- /app/dashboard/tests/test_dashboard_utils.py
- /app/economy/models.py
- /app/economy/tx.py
- /app/feeswapper/management/commands/feeSwapper.py
- /app/grants/models.py
- /app/grants/views.py
- /app/grants/management/commands/ingest_grant_txns.py
- /app/grants/management/commands/payout_round_noncustodial.py
- /app/grants/management/commands/payout_round.py
- /app/grants/management/commands/update_contributor_address.py
- /app/kudos/helpers.py
- /app/kudos/tasks.py
- /app/kudos/utils.py
- /app/kudos/views.py
- /app/kudos/management/commands/sync_kudos.py
- /app/ptokens/models.py
- /app/quadraticlands/views.py
- /app/revenue/views.py
- /app/townsquare/management/commands/payout_mini_clr.py

### Acceptance Criteria

When all scripts utilising web3.py have been checked and updated according to the migration guide and all regression tests pass

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.