-
Notifications
You must be signed in to change notification settings - Fork 118
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
Progression system #655
Progression system #655
Conversation
TODO:
|
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.
looks pretty good overall I only got some nitpicks
I don't know if the entire scoring thing is correct but who knows, really
I'm going to implement a notification thing to tell users that they can toggle it now, then I'm going to try to get a playtest going later today |
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.
Code looks very good, surely this will get merged soon
I just found EDIT: nevermind, respawn made this far too complex to be worth it |
Other than caedf37 the playtest picked up no issues with the PR. With that being said I am going to remove the label and mark as ready to merge. |
TODO LIST:
EDIT: Ok so this is basically done now, just going to do some playtesting and stuff before waiting 3 months for a merge