Skip to content

Minor SScommunication Refactor#6096

Merged
CheffieGithub merged 1 commit intoMonkestation:mainfrom
Ben10083:the_the_decrees
Apr 23, 2026
Merged

Minor SScommunication Refactor#6096
CheffieGithub merged 1 commit intoMonkestation:mainfrom
Ben10083:the_the_decrees

Conversation

@Ben10083
Copy link
Copy Markdown

@Ben10083 Ben10083 commented Apr 11, 2026

About The Pull Request

Refactor of SScommunication to remove unused elements + fix 'the the' when making an announcement

It seems currently we bypass SScommunication for most things, for now have it so making an announcement command in THROAT uses it, but it may be best to remove this subsystem entirely, as earlier it was ONLY used for making decrees (THROAT had its cooldown based on this, making it useless)

Why It's Good For The Game

'The the King Decrees" is bad

Changelog

🆑
spellcheck: Announcement for new decrees no longer uses 'the' twice
refactor: Refactor of SScommunication
/:cl:

Pre-Merge Checklist

  • You tested this on a local server.
  • This code did not runtime during testing.
  • You documented all of your changes.

@github-actions github-actions Bot added the Code Maintenance Fixing this mess one line at a time label Apr 11, 2026
@Ben10083 Ben10083 changed the title SScommunication Refactor Minor SScommunication Refactor Apr 15, 2026
@CheffieGithub CheffieGithub merged commit 1c19752 into Monkestation:main Apr 23, 2026
17 checks passed
github-actions Bot added a commit that referenced this pull request Apr 23, 2026
@Ben10083 Ben10083 deleted the the_the_decrees branch April 23, 2026 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Code Maintenance Fixing this mess one line at a time

Projects

Development

Successfully merging this pull request may close these issues.

3 participants