-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Added excludeAction param #38643
Added excludeAction param #38643
Conversation
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes. |
d13443e
to
43ffdde
Compare
As mentioned in the review, I assume it's due to |
43ffdde
to
c6ac7be
Compare
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.
Nice! Add a changelog, then it's good to go from my side 🚀
c6ac7be
to
f136eb1
Compare
@jvillafanez cool with it? |
Kudos, SonarCloud Quality Gate passed! |
My only "complain" is that the |
Description
Adding excludeActions parameter to fileActions registerDefaultActions
Motivation and Context
Imagine you write your own file list for a specific app, you want you to use all the default actions, but not the 'Rename' action.
Therefore you needed to define all the actions again in the app. This can now be overcome by passing ['Rename'] as excludeActions parameter.
How Has This Been Tested?
Manual UI Testing
Types of changes
Checklist: