-
-
Notifications
You must be signed in to change notification settings - Fork 6
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
add libraries visited to itemsholder #21
add libraries visited to itemsholder #21
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
d72d37f
to
fa9ae05
Compare
* Cache-based wrapper around localStorage. | ||
*/ | ||
public readonly itemsHolder: ItemsHoldr< | ||
Record<string, boolean>>; |
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.
This reminds me, I need to set up CI to catch formatting issues. And commit hooks. #22.
this.game.itemsHolder.setItem(other.title, true); | ||
const libraries = ["AngularLibrary", "ReactLibrary", "SolidLibrary", "SvelteLibrary", "VueLibrary" ]; | ||
if(libraries.every((library) => this.game.itemsHolder.getItem(library))) { | ||
window.open("https://hi.joshuakgoldberg.com", "_blank") |
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.
I still love this
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.
💯 !
Overview
What does this PR do? Why?
It adds the libraries a user has visited to itemsholder and if all libraries have been visited they get a nice surprise ;)
PR Checklist