-
Notifications
You must be signed in to change notification settings - Fork 0
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
Next level warning #61
base: master
Are you sure you want to change the base?
Conversation
…ext Level text and button bug
…ting prior levels
Hey Celebi I'm getting two errors that about code that is defined but never used: let incrLevel; if (!props.skippedWarning) function toggle() { Could u lmk what the purpose of this code is just to be safe? I did test the code though and it does work. It's just that merging is blocked until we remove these unused variables. |
I think your merge removed some code, i'll try to fix it and let you know |
I merged master into the branch and fixed the merge conflicts. |
Tysm Celebi! And sorry for messing up your code, I'll check it and then merge |
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.
Hey Celebi, these changes look great! Sorry for the late review of your pull reqeust.
Actually wait, I see that pressing the 'level tutorial' button no longer hides the level tutorial. I'm going to try to fix this real quick |
No description provided.