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

[NO GBP] Music Request Credits changed to CKEY #76751

Merged
merged 1 commit into from Jul 12, 2023

Conversation

SomeRandomOwl
Copy link
Contributor

About The Pull Request

I was requested by one of the game admins over in the TG discord to change the credit from character name to ckey.

Why It's Good For The Game

It's mostly a OOC thing, personally I don't know of a good enough reason to put here other than its a PR someone requested of me.

Changelog

🆑
fix: Music Request Credit shows CKEY instead of character name
/:cl:

@tgstation-server tgstation-server added the Fix Rewrites a bug so it appears in different circumstances label Jul 11, 2023
@SomeRandomOwl SomeRandomOwl changed the title Music Request Credits changed to CKEY [NO GBP] Music Request Credits changed to CKEY Jul 11, 2023
@tgstation-server tgstation-server added the GBP: No Update Prevent changes to the user's GBP balance from this PR label Jul 11, 2023
@SomeRandomOwl
Copy link
Contributor Author

I think adding no GBP for this is a smart idea, because its just something stupidly small

@BlueMemesauce
Copy link
Contributor

People were trying to tell you why to use ckeys in the last PR, but I'll try to explain the reasons to you again.
While it may work for crew with static names that people know them by, people with random names or animal names like "monkey (123)" won't be able to credit themselves.
The reason about OOC you said the admins told you is that it's mixing IC and OOC, which should be separate. It doesn't make sense for your characters name to show up like he requested it IC, since the music only exists OOC.

@SomeRandomOwl
Copy link
Contributor Author

SomeRandomOwl commented Jul 11, 2023

I know, i just have seen a few people who get upset from their CKEY being displayed to others and just eh i dunno, i was trying to appease both sides since i really personally dont care either way. But adding onto this with the week that my change was merged i have also seen people questioning why it shows character name too and just theres no winning whatever i do so i think just flat out doing CKEY is fine

@ZephyrTFA
Copy link
Contributor

This never should have been mob name and always ckey regardless.
I understand that certain downstreams want to have your ckey and who you are be not linked, but requesting music is literally an OOC thing.

@ZephyrTFA ZephyrTFA merged commit 2a77d87 into tgstation:master Jul 12, 2023
23 checks passed
comfyorange added a commit that referenced this pull request Jul 12, 2023
Jolly-66 pushed a commit to TaleStation/TaleStation that referenced this pull request Jul 12, 2023
…6750)

Original PR: tgstation/tgstation#76751
-----
## About The Pull Request

I was requested by one of the game admins over in the TG discord to
change the credit from character name to ckey.

## Why It's Good For The Game

It's mostly a OOC thing, personally I don't know of a good enough reason
to put here other than its a PR someone requested of me.

## Changelog

:cl:
fix: Music Request Credit shows CKEY instead of character name
/:cl:

---------

Co-authored-by: SomeRandomOwl <2568378+SomeRandomOwl@users.noreply.github.com>
@TheVekter
Copy link
Contributor

TheVekter commented Jul 18, 2023

I was the one who originally requested this and did so because I played a song where the person wanted to be credited and it said "Wraith (539) requested..." which had the lovely effect of telling literally everyone there was a cult.

Thankfully, the cult was already exceedingly loud at this point, but this PR fixes the issue wholesale. Thank you for doing this.

@SomeRandomOwl SomeRandomOwl deleted the music-naming branch July 23, 2023 22:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Rewrites a bug so it appears in different circumstances GBP: No Update Prevent changes to the user's GBP balance from this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants