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

Set allied/enemy groups and visualize via armor #6

Open
Maxopoly opened this issue May 16, 2020 · 3 comments
Open

Set allied/enemy groups and visualize via armor #6

Maxopoly opened this issue May 16, 2020 · 3 comments

Comments

@Maxopoly
Copy link
Member

Running /ally Hjaltland (maybe come up with a better command name) will use ProtocolLib to turn the chest plate and pants of all players on the group into green dyed armor and all of those not on the group into red dyed armor for the player. Properly respect if players are not wearing those armor pieces at all and tie usage to a permission, because it's effectively a bypass on listing members or member checking someone. Properly reset armor if its turned off.

@Maxopoly Maxopoly added this to Active To Dos - Unclaimed tasks looking for someone to fix them in Public Issue Board May 16, 2020
@xFier
Copy link

xFier commented May 17, 2020

I feel like this would be better tied into a namelayer overhaul where groups are replaced with some representation of a 'Nation', so a single nation can make a namelayer 'nation' and then define groups within that. It doesn't really make sense that a single group of people need 20+ groups.

@Maxopoly
Copy link
Member Author

I agree and a NameLayer overhaul is one of the things on my todo list, see CivClassic/NameLayer#16, CivClassic/NameLayer#18 and CivClassic/NameLayer#17

I think that's mostly unrelated to this feature though and if we were to ever change NameLayer in such a way, this feature could easily be adjusted

@KaloHG
Copy link

KaloHG commented May 6, 2021

it'd probably be beneficial to disable a check on a players allied/enemied status if the player is invisible. This should probably also link into JukeAlert in the sense that if an "enemied" or "allied" player hits a snitch they recieve a proper prefix that colors them according to their standing.

edit:
I wanted to include that implementing this should be done very carefully as to avoid this turning into an HCF styled system, in the sense where you do like, /team join and you can also view other team info. I think it'd also probably be helpful functionality to permit specific enemy teams to be specified instead of just all players.

Diet-Cola pushed a commit to Diet-Cola/Finale that referenced this issue Aug 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Public Issue Board
Active To Dos - Unclaimed tasks looki...
Development

No branches or pull requests

3 participants