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

Neutroswap Milestone 1&2 #160

Merged
merged 11 commits into from
Mar 19, 2024

Conversation

jonassunandar
Copy link
Contributor

Milestone Delivery Checklist

Link to the application pull request: #136 < please fill this in with the PR number of your application.

@jonassunandar jonassunandar changed the title Milestone 1&2 Neutroswap Milestone 1&2 Jan 8, 2024
@tedcahalleos tedcahalleos added the Milestone-Received The Milestone submission has been received by Operations label Jan 11, 2024
@nsjames
Copy link
Member

nsjames commented Jan 25, 2024

Hello!
I'm going through the materials and I found a few issues.

  • License file link is not working, and I don't see a license in the repository
  • The following contracts don't have their code verified:
    • 0x5f4b3D92bcb944a0e9B231C42D0615F8A27Bbcc0 (NitroPoolFactory)
    • 0x72076068Bd08f5D0AE541075f1E317b3B1d46d8f (FairAuctionFactory)
  • The analytics page link is broken

(I see some issue with the explorer not be able to load the 0x96A064DB6CA1D45e59756D3DAc7CD249D4D742030 address right now and have elevated internally)

Something seems to be broken with subgraph development links
image


Contract foundry tests are all passing 👍
image

@jonassunandar
Copy link
Contributor Author

Hi @nsjames thanks for pointing some issues

  • Analytics link is changed to https://analytics.neutroswap.io/. We change the analytics directly to mainnet to test the accuracy of The Graph and for clearer data
  • Currently V2 contracts are being audited now by SolidProof. No changes from the submitted contracts. Hopefully within this week we will update and verified all the contracts
  • License and The Graph issues, I'll bring it to the team

@nsjames
Copy link
Member

nsjames commented Jan 25, 2024

Thanks, looking forward to closing this one out 🙏 (I know we've been going back and forth for a while on tg)

@jonassunandar
Copy link
Contributor Author

  • License file link is not working, and I don't see a license in the repository
    Something seems to be broken with subgraph development links image

Hi James

  1. The graph issues: you need to query some variables to test it. For example to get available all pairs here
image
  1. We have added the licenses here: Neutroswap app and V2 contracts

Please let me know if you found other issues

@nsjames
Copy link
Member

nsjames commented Jan 30, 2024

Thanks, just waiting for those contracts to be verified and we're good to go.

@jonassunandar
Copy link
Contributor Author

Hi @nsjames
All the mainnet contracts are verified and audited by SolidProof. Please take a look at here

@tedcahalleos tedcahalleos removed the request for review from arhag February 7, 2024 18:34
@nsjames
Copy link
Member

nsjames commented Feb 14, 2024

Looking good, thanks!

@jonassunandar
Copy link
Contributor Author

Hi guys, is there be any specific areas of our milestone that require additional discussion or clarification, please do not hesitate to reach out @tedcahalleos @nsjames @wanderingbort @BenjaminGormanPMP

@tedcahalleos tedcahalleos added the Milestone-Approved The Milestone has been approved and will be paid. label Mar 19, 2024
@tedcahalleos
Copy link
Member

Approved.

@tedcahalleos tedcahalleos merged commit e7b520c into eosnetworkfoundation:main Mar 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Milestone-Approved The Milestone has been approved and will be paid. Milestone-Received The Milestone submission has been received by Operations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants