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

Group changes #2951

Merged
merged 7 commits into from
Jan 27, 2025
Merged

Group changes #2951

merged 7 commits into from
Jan 27, 2025

Conversation

AgaSpace
Copy link
Contributor

@AgaSpace AgaSpace commented Jun 6, 2023

I added "virtual" to all properties and methods.
Updated the R, G, B fields. These are now properties.
I added a property Color that gets and updates the values R,G,B but as Microsoft.Xna.Framework.Color.

The general idea of this PR is to allow developers to create their own groups, like SuperAdminGroup, but with more editing capabilities.

@pull-request-size pull-request-size bot added size/M and removed size/S labels Jun 6, 2023
@AgaSpace AgaSpace changed the title Updated Group Group changes Jun 6, 2023
TShockAPI/Group.cs Outdated Show resolved Hide resolved
@AgaSpace AgaSpace requested a review from Arthri June 8, 2023 12:25
Arthri
Arthri previously approved these changes Jun 15, 2023
@hakusaro hakusaro enabled auto-merge January 27, 2025 00:13
@hakusaro hakusaro merged commit 7058a9d into Pryaxis:general-devel Jan 27, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants