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

🪦 poa ganache #11367

Merged
merged 6 commits into from
Feb 13, 2025
Merged

🪦 poa ganache #11367

merged 6 commits into from
Feb 13, 2025

Conversation

alecsavvy
Copy link
Contributor

@alecsavvy alecsavvy commented Feb 13, 2025

Description

  • removes poa ganache from local dev
  • removes test-contracts workflow as it's not used, this contract hasn't changed in years anyway
  • modifies the identity startup code to not init web3 (poa) in libs when in dev
  • modifies relay to not initialize a web3 connection (poa) in dev, it does keep the acdc network id hardcoded to be able to correctly decode encoded abis. this is unfortunate

How Has This Been Tested?

run this
audius-compose down && npm run protocol && audius-cmd test

Copy link

changeset-bot bot commented Feb 13, 2025

⚠️ No Changeset found

Latest commit: d01e480

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@alecsavvy alecsavvy changed the title As/rip poa ganache 🪦 poa ganache Feb 13, 2025
@alecsavvy alecsavvy self-assigned this Feb 13, 2025
@alecsavvy alecsavvy enabled auto-merge (squash) February 13, 2025 17:23
@alecsavvy alecsavvy disabled auto-merge February 13, 2025 18:04
@alecsavvy alecsavvy merged commit 73d3ad1 into main Feb 13, 2025
12 of 13 checks passed
@alecsavvy alecsavvy deleted the as/rip-poa-ganache branch February 13, 2025 18:05
web3 = connectedWeb3.web3
config.acdcChainId = connectedWeb3.chainId.toString()
wallets = new WalletManager(web3)
if (config.environment !== "dev") {
Copy link
Contributor

Choose a reason for hiding this comment

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

Only real concern is that it means we have untestable production code paths. So I guess this means we catch any issues here in staging?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yep code path wouldn't change for staging

schottra added a commit that referenced this pull request Feb 13, 2025
* origin/main: (21 commits)
  Test listen streak challenge 2 listens in 1 day (#11373)
  Listen streak challenge fix 2 plays in one day (#11372)
  Randomize storage node ordering in StorageNodeSelector. (#11371)
  Disable endless listen streak challenge, delete rows (#11369)
  Fix user-link overflow (#11370)
  MRVR country fixes (#11118)
  [C-5270 C-5269 C-5268 C-5629] Comment layout improvements (#11360)
  🪦 poa ganache (#11367)
  Fix playlist invalid sort test (#11368)
  core block diff health check (#11365)
  Audius Protocol v0.7.90
  Hide endless challenge audio (#11364)
  [C-5769] Use batch fetching to normalize core entity hooks (#11355)
  entity manager through core on dev (#11359)
  fix lint (#11362)
  Fix plays indexing starvation (#11361)
  Revert "filter chain health on these nodes" (#11358)
  Tag audiusd with protocol releases to allow reverts (#11347)
  [C-5763 C-5603] Migrate profile collections to tan-query (#11285)
  [C-5767] Migrate related-artists to tan-query #2 (#11303)
  ...
schottra added a commit that referenced this pull request Feb 14, 2025
…ibles

* origin/main: (71 commits)
  [PAY-3881][PAY-3883] Add indexing/endpoints for separated collectibles data (#11287)
  [C5778] Update web profile page layout (#11374)
  Test listen streak challenge 2 listens in 1 day (#11373)
  Listen streak challenge fix 2 plays in one day (#11372)
  Randomize storage node ordering in StorageNodeSelector. (#11371)
  Disable endless listen streak challenge, delete rows (#11369)
  Fix user-link overflow (#11370)
  MRVR country fixes (#11118)
  [C-5270 C-5269 C-5268 C-5629] Comment layout improvements (#11360)
  🪦 poa ganache (#11367)
  Fix playlist invalid sort test (#11368)
  core block diff health check (#11365)
  Audius Protocol v0.7.90
  Hide endless challenge audio (#11364)
  [C-5769] Use batch fetching to normalize core entity hooks (#11355)
  entity manager through core on dev (#11359)
  fix lint (#11362)
  Fix plays indexing starvation (#11361)
  Revert "filter chain health on these nodes" (#11358)
  Tag audiusd with protocol releases to allow reverts (#11347)
  ...
audius-infra pushed a commit that referenced this pull request Feb 14, 2025
[c8a0e06] [PAY-3881][PAY-3883] Add indexing/endpoints for separated collectibles data (#11287) Randy Schott
[1b39bfc] Test listen streak challenge 2 listens in 1 day (#11373) Reed
[f854639] Listen streak challenge fix 2 plays in one day (#11372) Reed
[c62caf6] Disable endless listen streak challenge, delete rows (#11369) Reed
[4773a86] MRVR country fixes (#11118) Steve Perkins
[73d3ad1] 🪦 poa ganache (#11367) alecsavvy
[f74778f] Fix playlist invalid sort test (#11368) Isaac Solo
[1c65e9c] core block diff health check (#11365) alecsavvy
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants