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

less usage of generic rewards: Tacoma #77453

Merged

Conversation

Kamejeir
Copy link
Contributor

@Kamejeir Kamejeir commented Oct 30, 2024

Summary

None

Purpose of change

Bootstrapping the Tacoma ranch right now makes all the inhabitants indebted to you for life, but it's not as though that makes perfect sense: They can't just start giving away all their stock to you, because they still report to the Refugee Center to facilitate trading. Besides, most of them don't even have a shop running (the part that bothers me the most, as you get to look at taking their personal belongings when you don't really have any business doing so).

Describe the solution

Set "has_generic_rewards": false to most NPCs in Tacoma (exception is Scavenger Boss, because he does have a shop with enough value attached, while giving quests. Reward could feasibly come in the form of wares... you're practically selling gear to him with the quests. I just chose not to touch that because it involves arbitrary adjustment of the "value" value)

This being in draft: Setting up Added more believable rewards with the end of Doctor and Bartender quests (the foreman giving you cash for use in other shops)

Describe alternatives you've considered

Testing

When this is out of draft, I'll hopefully have done everything in regards to testing it working and adjusting dialogue options.

Did all the quests for the Foreman, Nurse, Doctor and Bartender (I couldn't figure out how to do the Pablo quest, but I will trust that the setting change that breaks for noone else, would not break with him). I mostly looked for the post-quest "How about some items as payment?" and did not see any such prompt for the questlines in question.

Tested the Foreman giving you cash, and I see no dialogue out of place (he only says the "Hello again" thing whenever there's an unclaimed reward)

Additional context

"Can you share some equipment?" on mission start is not something I've figured out how to avoid, but whether I should is something I have not yet pondered. In doing the Scrapper quests (for the Doctor) it seems such extraneous options don't exist, but I'm not perfectly how to achieve that, nevermind it not really being the point of this PR.

look karol, I'm deleting (rewards) by adding lines!

@github-actions github-actions bot added NPC / Factions NPCs, AI, Speech, Factions, Ownership [JSON] Changes (can be) made in JSON labels Oct 30, 2024
@github-actions github-actions bot added astyled astyled PR, label is assigned by github actions json-styled JSON lint passed, label assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Oct 30, 2024
@Kamejeir Kamejeir force-pushed the tacoman_rewards-generic_minus_scav branch from 3f1e98c to 0a71a70 Compare November 1, 2024 13:59
@Kamejeir Kamejeir marked this pull request as ready for review November 1, 2024 19:02
@Maleclypse Maleclypse merged commit 8907686 into CleverRaven:master Nov 2, 2024
25 of 40 checks passed
@Kamejeir Kamejeir deleted the tacoman_rewards-generic_minus_scav branch November 2, 2024 08:16
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 NPC / Factions NPCs, AI, Speech, Factions, Ownership
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants