Skip to content

core.inject_trigger: resolve trigger payload shadowing by renaming trigger param to trigger_name (backwards compatible) #5383

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

Merged
merged 13 commits into from
Oct 25, 2021

Conversation

cognifloyd
Copy link
Member

@cognifloyd cognifloyd commented Oct 7, 2021

This PR merges master into the code from #5335, and adds the changelog entry.

Closes #4641
Closes #5335

@pull-request-size pull-request-size bot added the size/S PR that changes 10-29 lines. Very easy to review. label Oct 7, 2021
@cognifloyd cognifloyd changed the title Trigger name collission core.inject_trigger: resolve trigger payload shadowing by renaming trigger param Oct 7, 2021
@cognifloyd cognifloyd changed the title core.inject_trigger: resolve trigger payload shadowing by renaming trigger param core.inject_trigger: resolve trigger payload shadowing by renaming trigger param to trigger_param Oct 7, 2021
@cognifloyd cognifloyd changed the title core.inject_trigger: resolve trigger payload shadowing by renaming trigger param to trigger_param core.inject_trigger: resolve trigger payload shadowing by renaming trigger param to trigger_name Oct 7, 2021
@cognifloyd cognifloyd changed the title core.inject_trigger: resolve trigger payload shadowing by renaming trigger param to trigger_name core.inject_trigger: resolve trigger payload shadowing by renaming trigger param to trigger_name (backwards compatible) Oct 7, 2021
@cognifloyd cognifloyd added this to the 3.6.0 milestone Oct 7, 2021
@cognifloyd cognifloyd enabled auto-merge October 7, 2021 21:13
@cognifloyd cognifloyd requested a review from winem October 18, 2021 18:22
@cognifloyd cognifloyd merged commit 5d5595e into master Oct 25, 2021
@cognifloyd cognifloyd deleted the trigger_name_collission branch October 25, 2021 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/S PR that changes 10-29 lines. Very easy to review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug: core pack: core.inject_trigger: trigger parameter masks trigger payload in rule setting
3 participants