-
Notifications
You must be signed in to change notification settings - Fork 70
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
Updated CustomItems & CustomRoles give commands to use default RAUtils.ProcessPlayerIdOrNamesList #179
Conversation
Also build your branch with Release not Debug because you wouldn't seen all the error to fix issue of compiling |
Alright. I'll do that as soon as I can. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some foreach loops can be simplified with LINQ extensions and PR must target dev branch
This reverts commit 410fb61.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm
@@ -18,6 +18,8 @@ namespace Exiled.CustomItems.Commands | |||
using Exiled.Permissions.Extensions; | |||
|
|||
using RemoteAdmin; | |||
using UnityStandardAssets.Effects; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
???
Description
Describe the changes
What is the current behavior? (You can also link to an open issue here)
Commands currently evaluate only
*
,all
, orPlayer.Id
What is the new behavior? (if this is a feature change)
Commands now evaluate players based on
RAUtils.ProcessPlayerIdOrNamesList
(eg@player.17.15
) as well as*
,all
, orPlayer.Id
.Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
No breaking changes
Other information:
Types of changes
Submission checklist
Patches (if there are any changes related to Harmony patches)
Other