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

Mentor PM Panel #22951

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Oblisk234
Copy link
Member

@Oblisk234 Oblisk234 commented Jan 9, 2025

Document the changes in your pull request

Turns out in #2906, there was a proc that never got a verb. Hence, with a few changes, Mentor PM Panel is now available to mentors.

For reference, I changed the mentor PM panel to only show mob names (and player ckeys when they're in the lobby) since mentors shouldn't really have the same info the admin PM verb would have.

Why is this good for the game?

MENTORHELP: Player (F): Hey I think [INSERT NAME HERE] is a new player, can you help them?

As the only mentor on, this is difficult to help with the current tools if you're a living player at the time. As a ghost, you can use Orbit menu, Right Click --> Mentor PM Mob, voila. As a living player, it becomes more of a weirder situation. You can either try to search for them in game (questionable, as you're looking for them for an OOC reason in an IC manner), or you can do nothing (Mentors do not do this, except by accident).

With a Mentor PM Panel, this solves the questionability of searching for someone in game for an mhelp, as well as providing a method of helping people when they normally wouldn't be able to (e.g. ghost roles).

Testing

image
image

Changelog

🆑
rscadd: Added a mentor verb for the Mentor PM Panel from the original mentor PR.
/:cl:

@Yogbot-13 Yogbot-13 added the Feature This adds new content to the game label Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature This adds new content to the game
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants