Skip to content

Prevents Non-Admin CO + Synthetic Medals and Ribbons#12178

Merged
realforest2001 merged 2 commits into
cmss13-devs:masterfrom
matttheficus:XORibbonLimit
May 6, 2026
Merged

Prevents Non-Admin CO + Synthetic Medals and Ribbons#12178
realforest2001 merged 2 commits into
cmss13-devs:masterfrom
matttheficus:XORibbonLimit

Conversation

@matttheficus
Copy link
Copy Markdown
Contributor

@matttheficus matttheficus commented May 6, 2026

About the pull request

Prevents XOs (or silly COs) from giving medals to COs or Synthetics.

Explain why it's good for the game

The Senators for both of the WLs above said to do it.

This does not affect medals or ribbons given via the Admin Medal Panel.

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑
code: added checks to prevent COs or Synthetics from being given medals or ribbons by non-admins.
/:cl:

@github-project-automation github-project-automation Bot moved this to Awaiting Review in Review Backlog May 6, 2026
@cmss13-ci cmss13-ci Bot added Code Improvement Make the code longer size/XS Denotes a PR that changes 0-19 lines, ignoring generated files. labels May 6, 2026
@Uristthedorf
Copy link
Copy Markdown
Contributor

Out of curiosity why prevent synths from getting non-admin medals?
Is it a WL metafriend issue or something?

@github-project-automation github-project-automation Bot moved this from Awaiting Review to Partially Approved / Mergeable in Review Backlog May 6, 2026
@realforest2001 realforest2001 added this pull request to the merge queue May 6, 2026
@Locardoc
Copy link
Copy Markdown

Locardoc commented May 6, 2026

Shouldn't the Liaison be unable to get medals from COs aswell, implying they're civilians?

Merged via the queue into cmss13-devs:master with commit dcdead9 May 6, 2026
27 checks passed
@github-project-automation github-project-automation Bot moved this from Partially Approved / Mergeable to Merged in Review Backlog May 6, 2026
cmss13-ci Bot added a commit that referenced this pull request May 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Code Improvement Make the code longer size/XS Denotes a PR that changes 0-19 lines, ignoring generated files.

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

4 participants