-
Notifications
You must be signed in to change notification settings - Fork 8
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
Update module1 to remove dependency on the Goerli test network #51
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
general changes look good, pls. see comment
tests/module1/run_e2e_pipeline.sh
Outdated
ALCHEMY_POLICY_ID="" | ||
ALCHEMY_API_KEY="" | ||
NFT_STORAGE_API_TOKEN="" | ||
RPC_ENDPOINT=https://eth-sepolia.g.alchemy.com/v2/tPlhuS4IDHbHHU7RBsZTUN9tciY4ojor |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please clean up - maybe we should move to e2e test case parameterisation to an env based approach to avoid checked in creds
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pipeline faces missing library issue right now ("@alchemy/aa-core") fix in progress
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
aa-core lib missing
Issue #, if available: Since the Goerli test network has been deprecated this workshop stopped working properly. There were hard coded variables making it impossible to run the workshop on other test networks such as Sepolia.
Description of changes: worked on removing the hard coded dependencies and move variables to AWS Systems Manager Parameter Store. Worked on fixing some issues with the frontend not being able to query the user wallet and NFT because of hard coded URL. Updated README.md files and integration tests to reflects those changes.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.