Skip to content

Filter map player home incident tags - #955

Merged
notfood merged 1 commit into
rwmt:devfrom
Kuinox:upstream/filter-map-player-home-incident-tags
Jul 27, 2026
Merged

Filter map player home incident tags#955
notfood merged 1 commit into
rwmt:devfrom
Kuinox:upstream/filter-map-player-home-incident-tags

Conversation

@Kuinox

@Kuinox Kuinox commented Jun 20, 2026

Copy link
Copy Markdown
Contributor

I don't know if there could be side effects.
I didn't found any bug after playing 4 hours with a friend.

@Kuinox
Kuinox marked this pull request as ready for review June 20, 2026 21:15
@notfood notfood added fix Fixes for a bug or desync. multifaction Bugs or issues only in multifaction mode. 1.6 Fixes or bugs relating to 1.6 (Not Odyssey). labels Jun 21, 2026
@notfood notfood moved this to In review in 1.6 and Odyssey Jun 21, 2026
@notfood
notfood merged commit 899e359 into rwmt:dev Jul 27, 2026
1 check passed
@github-project-automation github-project-automation Bot moved this from In review to Done in 1.6 and Odyssey Jul 27, 2026
CormacOConnor72 added a commit to CormacOConnor72/Multiplayer that referenced this pull request Jul 27, 2026
MultiplayerAsyncQuest.cs: kept this branch's TickQuests fix, which matches
upstream cbe74ae (rwmt#969) but uses an index loop over the live list instead of
snapshotting with ToList - no per-tick allocation, and a quest that ends
during its own tick is skipped this pass rather than ticked after ending.
Removals are deterministic, so both behaviors are the same on every client.

Also brings in rwmt#955 (another player faction's settlement no longer
advertises Map_PlayerHome to the current faction's storyteller), which
complements this branch's cross-faction incident suppression at the
generation end.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1.6 Fixes or bugs relating to 1.6 (Not Odyssey). fix Fixes for a bug or desync. multifaction Bugs or issues only in multifaction mode.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants