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

Added Jukebox (#26736) #802

Merged
merged 5 commits into from
Sep 2, 2024
Merged

Conversation

DJB1gYAPPA
Copy link
Contributor

@DJB1gYAPPA DJB1gYAPPA commented Aug 25, 2024

Description

Adds super cool super nice Jukebox to the bar! Ran off .ogg files, can use any royalty free song.


Media

![Example Media on]
Screenshot 2024-08-26 061947


Changelog

🆑 Kacey, Nova

  • add: Added Jukebox with basic songs.

* Added Jukebox, along with music for jukebox

* Fixed Jukebox meta.json copyright

* Removed songs I couldn't find a license for.

* Renamed files to solve check failures from spaces

* Added missing attributions.yml

* Fixed lack of description in Jukebox

* Jukebox is now constructable.

* Change Jukebox menu to FancyWindow

* Moved Jukebox messages out of jukebox component

* Removed Jukebox OnValueChanged.

* JukeboxComp now uses AutoGenerateComponentState

* Removed state code, since it's auto generated

* Fixed various Jukebox code to match conventions.

* Updated Standard.yml to match changed song list.

* fixes

* Jukebox workin

* Fix

* Polishing

* Finalising

* Revert

* bad

* jukey

* Reviews

* name

* Update submodule to 218.2.0

---------

Co-authored-by: iNVERTED <[email protected]>
@github-actions github-actions bot added Changes: Audio Changes any ogg files Changes: C# Changes any cs files Changes: Localization Changes any ftl files Changes: Sprite Changes any png or json in an RSI Changes: UI Changes any XAML files Changes: YML Changes any yml files labels Aug 25, 2024
Copy link
Contributor

github-actions bot commented Aug 25, 2024

RSI Diff Bot; head commit ecc44e7 merging into 6d53996
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/Structures/Machines/jukebox.rsi

State Old New Status
off Added
on Added
select Added

Edit: diff updated after ecc44e7

@DJB1gYAPPA DJB1gYAPPA marked this pull request as draft August 25, 2024 18:55
@DJB1gYAPPA DJB1gYAPPA marked this pull request as ready for review August 25, 2024 19:07
@VMSolidus
Copy link
Member

image

Test fail is legitimate.

Added: Jukebox circuit board
@DangerRevolution DangerRevolution added Size: 2-Large For large issues/PRs Priority: 1-Urgent Must be resolved immediately Type: Feature Creation of or significant changes to a feature Type: Port Brings something to here from another codebase labels Aug 26, 2024
@DangerRevolution
Copy link
Contributor

Fill out the PR please; add a changelog and (optionally) add a video of it working.

@DJB1gYAPPA
Copy link
Contributor Author

Fill out the PR please; add a changelog and (optionally) add a video of it working.

I can not get a local build to work for me, however, it works for everyone else. I wont be able to add a video of it working, or a snapshot of what it looks like in game

@FoxxoTrystan FoxxoTrystan self-requested a review August 29, 2024 22:34
Copy link
Member

@FoxxoTrystan FoxxoTrystan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please make a better Changlog, but overall.
Good job!

@VMSolidus VMSolidus merged commit b53ef76 into Simple-Station:master Sep 2, 2024
29 checks passed
SimpleStation14 added a commit that referenced this pull request Sep 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Audio Changes any ogg files Changes: C# Changes any cs files Changes: Localization Changes any ftl files Changes: Sprite Changes any png or json in an RSI Changes: UI Changes any XAML files Changes: YML Changes any yml files Priority: 1-Urgent Must be resolved immediately Size: 2-Large For large issues/PRs Status: Needs Review Someone please review this Type: Feature Creation of or significant changes to a feature Type: Port Brings something to here from another codebase
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants