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

Challenge #2 Completed #68

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

AlbuquerqueJC
Copy link

@AlbuquerqueJC AlbuquerqueJC commented Mar 19, 2024

Algorand Coding Challenge Submission

What was the bug?

There were errors with dependencies. "3 vulnerabilities (2 moderate, 1 critical)"

How did you fix the bug?

To address all issues, run:
"algokit bootstrap all"

"algokit bootstrap all" corrected all dependency issues and allowed the build to continue.

Console Screenshot:

image

npm audit fix corrected all dependency issues and allowed the build to continue.
@iskysun96
Copy link
Contributor

npm install works but there is an algokit command that installs all of the dependencies and also does other stuff to set up the project environment. What is that command? 👀

@AlbuquerqueJC
Copy link
Author

npm install works but there is an algokit command that installs all of the dependencies and also does other stuff to set up the project environment. What is that command? 👀

You're absolutely correct, I used "algokit bootstrap all".

@iskysun96 iskysun96 added the Approved ✅ Successfully completed the challenge! label Mar 19, 2024
@iskysun96
Copy link
Contributor

🎉 Congrats on solving the 2nd Algorand Coding Challenge!

I have labeled your PR approved so you can now claim your certificate.

You can claim your certificate of completion NFT here:
https://goplausible.xyz/claim/c45kmyrg5vxtmzd4pz1cefpq70e7rp8tx6dyjcve8w544f35kgf6155h5nq6dgj7q3gceg1214gqfjbysvqxsyf2bnp5eyc5rnhy230

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approved ✅ Successfully completed the challenge!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants