[Drafts] - Player's employment #6855
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
multimokia
added
awaiting code review
multimokia
reviewed
Jan 3, 2021
| ) | ||
|
|
||
| label monika_player_employment: | ||
| if mas_event_mailbox.is_monika_initiated_mode(): |
There was a problem hiding this comment.
Just also an FYI to leave this out for now until we get the talk menu awareness stuff merged into upstream MAS, since
a) These aren't necessarily final
b) These don't exist in content right now
There was a problem hiding this comment.
I'll take these out in my next draft, no problem. :D
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is a beginning draft for some player variables regarding work, I'm also working on an "are you in school" version.
This first iteration contains a lot of menu options, but I tried to keep them as a pretty wide umbrella. I might condense employment and freelance into one option.
I would like to open this for discussion, as I'm not certain if it currently suits what we want for other dialogue options.
Todos;