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

New Additions and Commands #189

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Yuzurukyun
Copy link

These are slight additions, but they mean well, especially the /getmusic command that everyone is looking forward to. It gives options for those who actually like the old feature of music changing per area before it was omitted.

Repurposed the function to work despite the server giving it the big "NO".
And added an attribute to force music playing on the spot.
The function exists to change music upon entering an area with the auto_getmusic attribute or change_to_getmusic are True.
Okay, this is a big one.
-> Added Three Music Related Commands: /getmusic, /autogetmusic, /setgetmusic
-> Added One Summon Related Command: /summon_all
-> Quality of Life Changes for /look

- /getmusic: This command will play the current area's music. It will not play anything if there's no music playing.
- /autogetmusic: A client toggle that, if True, will play the current area's music upon entering any area. It is False by default.
- /setgetmusic: An area toggle that, if True, will play the current area's music upon entering the area. It is False by default and works per area.

- /summon_all: Works similar to /summon, but will take EVERYONE BUT STAFF within the hub's areas and move them to the argument's area. 

- /look: Now writes Area ID before Area Name. 
Example: === Look results for Area <area_id>: <area_name> ===
@TrickyLeifa
Copy link

What is this PR even supposed to do? What do each of these additions/commands even do?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants