Skip to content

Conversation

JohnFulpWillard
Copy link
Member

About The Pull Request

This isn't the song that plays at the end of a round (which is actually just the roundstart song played again), but the little few second theme that plays when a server reboots.
This sets it to use the same thing title music does.

Why It's Good For The Game

  • Allows admins to add/remove songs without having to manually VV edit it in round or go and edit code
  • Removes more hardcoded things that shouldn't be hardcoded
  • Makes roundstart and roundend's songs more consistent with eachother.

@tgstation-server tgstation-server added the Sound Oh god my ears, they bleed, did you normalise the volume? label Sep 19, 2021
@JohnFulpWillard
Copy link
Member Author

im mad because this works locally so idk why checks are failing

@JohnFulpWillard JohnFulpWillard marked this pull request as draft September 19, 2021 20:12
@Mothblocks
Copy link
Member

CC @MrsTonedOne -- files being moved into config

@JohnFulpWillard JohnFulpWillard marked this pull request as ready for review September 20, 2021 11:47
@Mothblocks Mothblocks added the Do Not Merge You must have really upset someone label Sep 22, 2021
@Mothblocks
Copy link
Member

I'm going to wait until @MrsTonedOne has confirmed these are all moved over

@Mothblocks
Copy link
Member

or maybe do it myself later

@github-actions
Copy link
Contributor

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself

@github-actions github-actions bot added the Stale Even the uncaring universe rejects you, why even go on label Sep 29, 2021
@JohnFulpWillard
Copy link
Member Author

@Mothblocks Any update on this?

@Mothblocks
Copy link
Member

mso is busy with forums so i'll have to do it myself

@Mothblocks
Copy link
Member

actually i dont even think i have the ability to network link stuff

@github-actions github-actions bot removed the Stale Even the uncaring universe rejects you, why even go on label Oct 2, 2021
@MrsTonedOne
Copy link
Contributor

actually i dont even think i have the ability to network link stuff

image

@Mothblocks
Copy link
Member

I'll look into it, to learn if nothing else

@Mothblocks Mothblocks self-assigned this Oct 2, 2021
@Mothblocks
Copy link
Member

Done

@Mothblocks Mothblocks removed the Do Not Merge You must have really upset someone label Oct 5, 2021
@Mothblocks Mothblocks merged commit 9693fb4 into tgstation:master Oct 5, 2021
github-actions bot added a commit that referenced this pull request Oct 5, 2021
@JohnFulpWillard JohnFulpWillard deleted the roundend-song-config branch February 20, 2022 07:46
buffyuwu pushed a commit to The-Final-Nights/The-Final-Nights that referenced this pull request Apr 17, 2025
<!-- Write **BELOW** The Headers and **ABOVE** The comments else it may
not be viewable. -->
<!-- You can view Contributing.MD for a detailed description of the pull
request process. -->

## About The Pull Request

<!-- Describe The Pull Request. Please be sure every change is
documented or this can delay review and even discourage maintainers from
merging your PR! -->

Ports the following TG station PRs:
- [x] tgstation/tgstation#59667
- [x] tgstation/tgstation#61573
- [x] tgstation/tgstation#67455
- [x] tgstation/tgstation#72805
- [x] tgstation/tgstation#74170
- [x] tgstation/tgstation#74650
- [x] tgstation/tgstation#75691
- [x] tgstation/tgstation#76453
- [x] tgstation/tgstation#76751
- [x] tgstation/tgstation#81135
- [x] tgstation/tgstation#81923
- [x] tgstation/tgstation#82152
- [x] tgstation/tgstation#85953
- [x] tgstation/tgstation#85967
- [x] tgstation/tgstation#89470
- [x] tgstation/tgstation#89777
- [x] tgstation/tgstation#90330


## Why It's Good For The Game

<!-- Argue for the merits of your changes and how they benefit the game,
especially if they are controversial and/or far reaching. If you can't
actually explain WHY what you are doing will improve the game, then it
probably isn't good for the game in the first place. -->

jukebox fixes + audio stuff is good...

## Testing Photographs and Procedure
<!-- Include any screenshots/videos/debugging steps of the modified code
functioning successfully, ideally including edge cases. -->
<!-- You can uncomment line 1 @ _maps/_basemap.dm to boot up a test map
that loads much faster. -->
<details>
<summary>Screenshots&Videos</summary>



https://github.com/user-attachments/assets/c5ef5f9c-0f8b-4b16-a9aa-e6a36c04443f



</details>

## Changelog

<!-- If your PR modifies aspects of the game that can be concretely
observed by players or admins you should add a changelog. If your change
does NOT meet this description, remove this section. Be sure to properly
mark your PRs to prevent unnecessary GBP loss. You can read up on GBP
and its effects on PRs in the tgstation guides for contributors. Please
note that maintainers freely reserve the right to remove and add tags
should they deem it appropriate. You can attempt to finagle the system
all you want, but it's best to shoot for clear communication right off
the bat. -->

:cl: XeonMations, JJRcop
add: Added a new verb that players can request audio from admins with
add: Admins can now play internet links.
config: More config changes related to roundstart and roundend music.
/:cl:

<!-- Both :cl:'s are required for the changelog to work! You can put
your name to the right of the first :cl: if you want to overwrite your
GitHub username as author ingame. -->
<!-- You can use multiple of the same prefix (they're only used for the
icon ingame) and delete the unneeded ones. Despite some of the tags,
changelogs should generally represent how a player might be affected by
the changes rather than a summary of the PR's contents. -->

---------

Co-authored-by: Jonathan Rubenstein <jrubcop@gmail.com>
Co-authored-by: san7890 <the@san7890.com>
Co-authored-by: Kylerace <kylerlumpkin1@gmail.com>
Co-authored-by: SomeRandomOwl <2568378+SomeRandomOwl@users.noreply.github.com>
Co-authored-by: Fikou <23585223+Fikou@users.noreply.github.com>
Co-authored-by: Aleksej Komarov <stylemistake@gmail.com>
Co-authored-by: Watermelon914 <37270891+Watermelon914@users.noreply.github.com>
Co-authored-by: MrMelbert <51863163+MrMelbert@users.noreply.github.com>
Co-authored-by: John Willard <53777086+JohnFulpWillard@users.noreply.github.com>
Co-authored-by: PeriodicChaos <44913068+PeriodicChaos@users.noreply.github.com>
Co-authored-by: EnnyDaiz <160051896+EnnyDaiz@users.noreply.github.com>
Co-authored-by: harry <me@harryob.live>
Co-authored-by: harryob <55142896+harryob@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Sound Oh god my ears, they bleed, did you normalise the volume?
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants