-
Notifications
You must be signed in to change notification settings - Fork 7
fix: Some upgrades erc725.js and some minor npms for security fixes #124
Conversation
@dzbo NOTE: When I ran the test locally the 404 page was a few pixels off. Not sure what that's about, but I am assuming it's just a local thing, but FYI. |
Deployed with Cloudflare Pages ☁️ 🚀 🆗 |
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.
Yes except for the API keys?!
.env.example
Outdated
ALGOLIA_APPLICATION_ID=YOUR_ALGOLIA_APPLICATION_ID | ||
ALGOLIA_API_KEY=YOUR_ALGOLIA_API_KEY | ||
ALGOLIA_APPLICATION_ID=YHFN1WRCR5 | ||
ALGOLIA_API_KEY=5981bf15ffc7630353478168f2574228 |
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.
??? Is this supposed to be public?!
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.
Argh, how did that get in there.
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.
Fixing it ended up in the example and not in .env.
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.
Removed. It's only inside of this PR note. I removed it from git history and the repo is not public.
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.
It's actually a public key for search anyways, but better not in the repo.
Description
Upgrades for erc725.js
Upgrades for some minors npms to capture security fixes.