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

Refactor patreon+twitch data to be linked to /datum/player_details than /client #4408

Merged
merged 2 commits into from
Dec 5, 2024

Conversation

Absolucy
Copy link
Member

@Absolucy Absolucy commented Nov 29, 2024

About The Pull Request

see title

this shouldn't have any player-facing changes, this is simple refactoring the code to be cleaner and more resilient.

Why It's Good For The Game

Avoids some runtimes if player disconnects early, and avoids some repeated operations (mostly SQL queries) during reconnects.

Changelog

🆑
refactor: Refactored how some internal stuff regarding Patreon and Twitch ranks are handled.
/:cl:

@Absolucy Absolucy added the do not merge don't merge this ffs label Nov 29, 2024
@Absolucy Absolucy force-pushed the player-details-refactor branch from 6a51b45 to 3f59a52 Compare November 29, 2024 04:34
@Absolucy Absolucy force-pushed the player-details-refactor branch from 3f59a52 to 3b7496c Compare December 2, 2024 03:36
@Absolucy Absolucy removed the do not merge don't merge this ffs label Dec 2, 2024
@Absolucy Absolucy marked this pull request as ready for review December 2, 2024 16:06
This was referenced Dec 2, 2024
@Absolucy Absolucy merged commit d61b73e into Monkestation:master Dec 5, 2024
25 checks passed
@Absolucy Absolucy deleted the player-details-refactor branch December 5, 2024 18:08
github-actions bot added a commit that referenced this pull request Dec 5, 2024
Absolucy added a commit to Absolucy/Monkestation that referenced this pull request Dec 17, 2024
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.

1 participant