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

Added A New level #3

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Conversation

jeremyfelder
Copy link

Added a new level.
Also added base64 encoded hints at the bottom in case someone gets stuck

@jeremyfelder
Copy link
Author

@fvictorio Let me know if I can do something to help get this merged

@fvictorio
Copy link
Owner

Ugh, sorry for not reviewing this yet @jeremyfelder, and thanks a lot for sending a new level. I promise I'll review it this weekend.

@fvictorio
Copy link
Owner

Hey @jeremyfelder, I need to take a deeper look at this but... it seems like it can be solved with an empty data and 0 value? That is, it works by default 😅 Maybe that's what you were going after but my guess is that it wasn't?

@jeremyfelder
Copy link
Author

jeremyfelder commented Mar 6, 2022

Hey @jeremyfelder, I need to take a deeper look at this but... it seems like it can be solved with an empty data and 0 value? That is, it works by default 😅 Maybe that's what you were going after but my guess is that it wasn't?

@fvictorio 😅 That was not intended indeed, great catch. I added some additional checks to prevent this and other invalid inputs

Added hints to the puzzle_X.json and attempts the user should be prompted on.
@jeremyfelder
Copy link
Author

@fvictorio I updated the new level to the refactored style and added hint prompts on level defined attempts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants