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

Not too generous Smokes #77225

Merged
merged 1 commit into from
Oct 21, 2024
Merged

Conversation

gettingusedto
Copy link
Contributor

Summary

None

Purpose of change

Whenever you complete a quest for smokes, you can ask smokes for item rewards, which gives the player basically 500 barter value credit for buying his stuff, which is rather on odds with the merch he gave beforehand. I feel like this is an oversight and i'm fixing it.

Describe the solution

give "has_generic_rewards": false to smokes' mission jsons.

Describe alternatives you've considered

Letting the oversight be.

Increasing the merch pay out for the back bay mission in particular, but that's a question for someone else to discuss.

Testing

Screenshot_20241021_162856

Additional context

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Missions Quests and missions json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions labels Oct 21, 2024
@dseguin dseguin merged commit f3d38e2 into CleverRaven:master Oct 21, 2024
24 of 25 checks passed
@gettingusedto gettingusedto deleted the not-that-generous branch October 21, 2024 14:12
@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Oct 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions Missions Quests and missions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants