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.

Sign up for GitHub

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

Jump to bottom

[Admin] Rename customTrigger param to snake_case notation #17436

Merged
merged 1 commit into from
Nov 7, 2024
Merged

[Admin] Rename customTrigger param to snake_case notation #17436

merged 1 commit into from
Nov 7, 2024

Conversation

GSadee
Copy link
Member

@GSadee GSadee commented Nov 7, 2024

Q A
Branch? 2.0
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Related tickets related to #17338
License MIT

@GSadee GSadee added DX Issues and PRs aimed at improving Developer eXperience. Admin AdminBundle related issues and PRs. labels Nov 7, 2024
@GSadee GSadee requested review from a team as code owners November 7, 2024 13:43
@GSadee GSadee merged commit c9fb267 into Sylius:2.0 Nov 7, 2024
31 checks passed
@GSadee GSadee deleted the admin-rename-custom-trigger-param branch November 7, 2024 13:55
@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 7, 2024 •
edited
Loading

Bunnyshell Preview Environment deployed

It will be automatically stopped in 4 hours.

Use the command /bns:start to start it if it's stopped.

Component Endpoints
mailhog https://mailhog-gx7rmr.bunnyenv.com/
php https://store-gx7rmr.bunnyenv.com/

Available commands:

  • /bns:stop to stop the environment
  • /bns:deploy to redeploy the environment
  • /bns:delete to remove the environment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@NoResponseMate NoResponseMate NoResponseMate approved these changes

Assignees
No one assigned
Labels
Admin AdminBundle related issues and PRs. DX Issues and PRs aimed at improving Developer eXperience.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@GSadee @NoResponseMate