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

Fix comparison in possible_lead_den_events for status='former ClanCat' #3057

Merged
merged 1 commit into from
Nov 29, 2024

Conversation

bakermor
Copy link
Contributor

About The Pull Request

In possible_lead_den_events there is a comparison between the cat's status.casefold() ("former clancat") and the event's allowed statuses ("former Clancat") which prevents the event from being added to possible_events.

Linked Issues

Fixes #3056
Also references PR #3025

Proof of Testing

image
Failure Outcome:
image
Success Outcome:
image

Generated Events Before:
image
Generated Events After:
image

Changelog/Credits

@j-gynn
Copy link
Contributor

j-gynn commented Nov 28, 2024

oh.
*screams internally that somehow that slipped past me*
at least it's not my worst clangen nonsense i've ever pushed to main...!

@scribblecrumb
Copy link
Collaborator

Thanks for the bugfix! Are you part of our discord server? If you'd like and if you plan to continue contributing, you're eligible to receive dev roles there and access the dev channels. It's not a requirement, though! And if you're welcome to continue contributing without being on the discord.

@scribblecrumb scribblecrumb merged commit 7235999 into ClanGenOfficial:development Nov 29, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Game crashes when inviting former Clancat
3 participants