Skip to content

feat: player factions - #184

Merged
Jesus-QC merged 4 commits into
northwood-studios:devfrom
LumiFae:sides
Jun 26, 2025
Merged

feat: player factions#184
Jesus-QC merged 4 commits into
northwood-studios:devfrom
LumiFae:sides

Conversation

@LumiFae

@LumiFae LumiFae commented Jun 14, 2025

Copy link
Copy Markdown
Member

Simple, just adds a Side property to player that allows people to easily tell if people are on the same side, i.e. can't attack each other if FF is off.

@LumiFae

LumiFae commented Jun 14, 2025

Copy link
Copy Markdown
Member Author

this now uses faction instead of a new enum

@LumiFae LumiFae changed the title feat: player sides feat: player factions Jun 14, 2025
@Axwabo

Axwabo commented Jun 15, 2025

Copy link
Copy Markdown
Contributor

Consider getting the faction based on the team of the role base for consistency as the Team property uses the role base, too
(What if the role base's team doesn't match that of the role type)

@Jesus-QC
Jesus-QC merged commit 838e39e into northwood-studios:dev Jun 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants