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

feat: add Scroll Alpha Testnet #112

Merged
merged 1 commit into from
Apr 7, 2023
Merged

Conversation

dghelm
Copy link
Contributor

@dghelm dghelm commented Mar 27, 2023

Hoping to add deployed contracts on Scroll Alpha Testnet.

Original contract deployment output viewable here.

Note: I'm unsure why running yarn generate also modified polygon and avax files, so please provide input on if this was an error on my end.

Edit:
This is to follow through on this Aave Governance Proposal and progress to this point has been aided by help from @foodaka

@sakulstra
Copy link
Collaborator

Note: I'm unsure why running yarn generate also modified polygon and avax files, so please provide input on if this was an error on my end.

it's currently a problem with the ci not updating automatically when irs change onchain.
It's fine to update here <3

Will get back soon to you as I have to follow up on what this is exactly - am i correct to assume it's related to: https://governance.aave.com/t/deploy-aave-v3-on-scroll-testnet/11861 ?

@sakulstra
Copy link
Collaborator

@dghelm had a closer look and upon a bit of inspection it seems like non of the contracts (that i checked) are verified.
I assume this is the correct blockexplorer: https://blockscout.scroll.io/address/0x1f95008723dACb3EDab2446B1d03204928EFfdE5/contracts#address-tabs ?

At least for now i'd hold off from adding non verified contracts. Also i'm not 100% sold yet on adding networks not supported on the existing uis. We'll need to discuss internally and eventually adjust the readme, guidlines.
Sorry for the work you spend on this pr.

@dghelm
Copy link
Contributor Author

dghelm commented Mar 30, 2023

Will get back soon to you as I have to follow up on what this is exactly - am i correct to assume it's related to: https://governance.aave.com/t/deploy-aave-v3-on-scroll-testnet/11861 ?

This is correct. Sorry to not call it out in the PR description. Edited.

At least for now i'd hold off from adding non verified contracts. Also i'm not 100% sold yet on adding networks not supported on the existing uis. We'll need to discuss internally and eventually adjust the readme, guidlines.

You've identified the correct block explorer -- and I understand the hesitation to trust a self-hosted, open-source "source of truth" here over something more centrally protected like Etherscan.

That said, Blockscout supports the same verification APIs as Etherscan, but unfortunately is seeing an issue around correctly indexing (and verifying) contracts created through factories. I have run the verifier on what I can at this point though and will keep you posted as we work through issues with Blockscout.

@dghelm
Copy link
Contributor Author

dghelm commented Apr 5, 2023

Also i'm not 100% sold yet on adding networks not supported on the existing uis.

I'm just realizing I may have misunderstood what you mean here. Are you suggesting the library wouldn't support projects that aren't included in Aave's frontend UI repo? Because @foodaka asked us to make the PR here so that we could get into that repo since it uses this library as a dependency. Might be a bit of a chicken and an egg issue :)

@sakulstra sakulstra merged commit 8c44ce6 into bgd-labs:main Apr 7, 2023
@dghelm
Copy link
Contributor Author

dghelm commented Apr 10, 2023

Thanks for the approval! Just to document, Scroll's Blockscout can now verify all contracts except for FlashLoanLogic, L2Pool-Implementation and PoolConfiguration-Implementation. Looks like Optimism Goerli verifies the latter 2, so we'll keep looking into how to address that.

@sakulstra As for npm library release cycles, is that something external contributors can help with, or is it best to just keep an eye on releases?

@sakulstra
Copy link
Collaborator

@dghelm , it's released now. Wanted to include some other additions as well.

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.

None yet

2 participants