-
-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Support wildcard classname in cloud triggers #7360
Support wildcard classname in cloud triggers #7360
Conversation
Codecov Report
@@ Coverage Diff @@
## master #7360 +/- ##
==========================================
- Coverage 93.92% 93.89% -0.03%
==========================================
Files 181 181
Lines 13195 13198 +3
==========================================
- Hits 12393 12392 -1
- Misses 802 806 +4
Continue to review full report at Codecov.
|
I tried to create class with the name Also I didn't change addTrigger function since it only checks for pre-defined classes. |
This test fails. |
Note for reviewers: there is an ongoing discussion in #7361 about how the feature should be implemented. |
|
Issue Description
This pull request solves this problem
https://community.parseplatform.org/t/support-wildcard-classname-in-cloud-triggers/1584
Related issue: #7361
Approach
If user set trigger for class "*", that trigger will be called for every class unless class has its own trigger.
TODOs before merging