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

Faction relationships can be modified through dialogue #77371

Merged

Conversation

RenechCDDA
Copy link
Member

@RenechCDDA RenechCDDA commented Oct 27, 2024

Summary

Features "Faction relationships can be modified through dialogue"

Purpose of change

There are cases where it's inappropriate for a player to interact with a NPC faction's things, but they may later gain permission. e.g. as a reward for performing some task.

There is currently no way to actually grant this permission.

Describe the solution

Make a way to grant these permissions.

Add a new dialogue effect which allows any relationship rule to be set/unset

Had to unfuck some lumbermill stuff while I was here. It was previously locked with no way in (lol). Unlocked the outer gate, added a dialogue option to ask for a gate to be opened. Does NOT actually give you permissions, since otherwise you'd be able to steal their gas.

Describe alternatives you've considered

Make some dialogue access for std::bitset<>flip()?

Testing

It works, it persists through load

Additional context

Hooking it up in a few more places in json, then undraft Done

@github-actions github-actions bot added <Documentation> Design documents, internal info, guides and help. NPC / Factions NPCs, AI, Speech, Factions, Ownership [C++] Changes (can be) made in C++. Previously named `Code` [Markdown] Markdown issues and PRs Player Faction Base / Camp All about the player faction base/camp/site <Enhancement / Feature> New features, or enhancements on existing json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions labels Oct 27, 2024
@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Oct 27, 2024
-Outer fence is no longer permalocked with no way in
-You can ask for a gate to be opened.
@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Map / Mapgen Overmap, Mapgen, Map extras, Map display labels Oct 27, 2024
@RenechCDDA RenechCDDA marked this pull request as ready for review October 27, 2024 09:05
@Maleclypse Maleclypse merged commit 1b45a75 into CleverRaven:master Oct 31, 2024
20 of 27 checks passed
@RenechCDDA RenechCDDA deleted the dynamic_dialogue_fac_relations branch October 31, 2024 09:31
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 [C++] Changes (can be) made in C++. Previously named `Code` <Documentation> Design documents, internal info, guides and help. <Enhancement / Feature> New features, or enhancements on existing [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions Map / Mapgen Overmap, Mapgen, Map extras, Map display [Markdown] Markdown issues and PRs NPC / Factions NPCs, AI, Speech, Factions, Ownership Player Faction Base / Camp All about the player faction base/camp/site
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants