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

admin character panel #474

Draft
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

lunarcomets
Copy link
Contributor

@lunarcomets lunarcomets commented Nov 18, 2024

About the PR

adds a character panel for admins to use to look up a player's current character's description, records, and event preferences to help with game master events. confidential records can come in another pr

Why / Balance

because currently looking at a player's preferences is pretty difficult due to the requirement of toolshed, this also consolidates most of a current character's information into one panel for an admin to get a quick overview of.

Technical details

to-do: this

Media

to-do: also this

Requirements

  • I have read and I am following the Pull Request Guidelines. I understand that not doing so may get my pr closed at maintainer’s discretion
  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

for admins: adds the character panel to more easily see details about a player's current character

@dffdff2423 dffdff2423 linked an issue Nov 19, 2024 that may be closed by this pull request
@lunarcomets lunarcomets changed the title Character panel and (hopefully) confidential records admin character panel Nov 19, 2024
@lunarcomets
Copy link
Contributor Author

@dffdff2423 regarding the link of the issue: as i'm working on this, it's currently going towards the panel only displaying the player's current in-round character, not being able to select from all of a player's profiles as is ideal in the issue. is that something that can just be added later, and would the current character data (which should be gotten from the db either way) be good enough for now?

@dffdff2423
Copy link
Collaborator

Yep! That is perfectly fine. I don't expect adding the ability to view other profiles will be that hard to add if you are already fetching them from the DB.

@dffdff2423 dffdff2423 linked an issue Dec 17, 2024 that may be closed by this pull request
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

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

Successfully merging this pull request may close these issues.

Character Panel Replace Admin Records Console with a general player info page.
2 participants