-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Seperate firebase from accounts.ts to firebase.ts #1638
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@GalaxyGamingBoy is attempting to deploy a commit to the Hack Club Team on Vercel. A member of the Team first needs to authorize it. |
Plagiarism Checker failing because of multiple js files being inputted |
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 have you tested and made sure the application works properly across?
games/Countdown_Golf.js
Outdated
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 would be ideal if you don't include unrelated changes in a PR
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.
The change of the games was due to having my main branch as the origin. The change was also warranted because without it the build failed. It would be more ideal if sprig didn't crash
why was this closed? @GalaxyGamingBoy |
I am fixing the commit, missed some places and got minor conflicts |
closes #1596