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

Serverside music #608

Merged
merged 9 commits into from Feb 3, 2022
Merged

Serverside music #608

merged 9 commits into from Feb 3, 2022

Conversation

ryanm2711
Copy link
Collaborator

@ryanm2711 ryanm2711 commented Jan 29, 2022

Modifications to the music system to make it serverside - meaning others can hear what you are listening to.

Closes #322

ryanm2711 and others added 6 commits January 27, 2022 13:04
Music is now played serverside, meaning that other players can hear the music inside the TARDIS.
Changed some things around to keep it future-proof for any further changes to the music screen.
I forgot to move one line over to the networked message
@parar020100 parar020100 added this to In progress / closed / finished in ParAr's personal sorting Jan 30, 2022
- merged cl_music and sv_music into sh_music
- only network music stop when explicitly pressing stop
- fix error when pressing play with no music selected
- remove unneeded function in SendMessage call
- removed unnecessary PlayerEnter hook
- added option for music to stop on exit
@MattJeanes MattJeanes merged commit fb384ed into dev Feb 3, 2022
@MattJeanes MattJeanes deleted the serverside-music branch February 3, 2022 01:35
@MattJeanes MattJeanes mentioned this pull request Feb 3, 2022
@parar020100 parar020100 mentioned this pull request Feb 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
ParAr's personal sorting
In progress / closed / finished
Development

Successfully merging this pull request may close these issues.

Networked Music
3 participants