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

Updating order with a custom controller action #3

Merged
merged 14 commits into from
Jun 17, 2024
Merged

Updating order with a custom controller action #3

merged 14 commits into from
Jun 17, 2024

Conversation

Zales0123
Copy link
Collaborator

This PR does not provide any additional functionalities but refactors the current logic to get rid of the decorated resource update handler and artificial additional state machine transition 🖖

@Zales0123 Zales0123 added the enhancement New feature or request label Jun 13, 2024
@Zales0123 Zales0123 force-pushed the custom-action-order-update branch 3 times, most recently from a688a36 to 35aecaf Compare June 13, 2024 17:01
@Zales0123 Zales0123 force-pushed the custom-action-order-update branch from 35aecaf to 52b750e Compare June 13, 2024 17:09
loevgaard
loevgaard reviewed Jun 13, 2024
View reviewed changes
psalm.xml Show resolved Hide resolved
loevgaard
loevgaard reviewed Jun 13, 2024
View reviewed changes
composer.json Outdated Show resolved Hide resolved
loevgaard
loevgaard reviewed Jun 13, 2024
View reviewed changes
composer.json Outdated Show resolved Hide resolved
loevgaard
loevgaard reviewed Jun 13, 2024
View reviewed changes
composer.json Outdated Show resolved Hide resolved
@codecov Codecov
Copy link

codecov bot commented Jun 14, 2024

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@Zales0123 Zales0123 force-pushed the custom-action-order-update branch from 2c856b3 to 3ceb3bd Compare June 14, 2024 10:40
loevgaard
loevgaard reviewed Jun 14, 2024
View reviewed changes
infection.json.dist Outdated Show resolved Hide resolved
loevgaard
loevgaard reviewed Jun 14, 2024
View reviewed changes
infection.json.dist Outdated Show resolved Hide resolved
@Zales0123 Zales0123 force-pushed the custom-action-order-update branch 2 times, most recently from 9eb2cd7 to dc3cbab Compare June 14, 2024 11:21
@Zales0123 Zales0123 force-pushed the custom-action-order-update branch from dc3cbab to 53b1d8a Compare June 14, 2024 11:29
@Zales0123 Zales0123 force-pushed the custom-action-order-update branch from 4e730ef to 2c397ad Compare June 14, 2024 12:22
@Zales0123 Zales0123 force-pushed the custom-action-order-update branch from 75b861c to 7014a6e Compare June 14, 2024 13:14
@Zales0123 Zales0123 force-pushed the custom-action-order-update branch from a7ca2dc to 85856bd Compare June 14, 2024 13:19
@Zales0123 Zales0123 force-pushed the custom-action-order-update branch from f6de861 to f985348 Compare June 17, 2024 06:40
@Zales0123 Zales0123 force-pushed the custom-action-order-update branch from f985348 to 2e795bd Compare June 17, 2024 07:57
@Zales0123 Zales0123 mentioned this pull request Jun 17, 2024
Merged
@Zales0123 Zales0123 merged commit 609f8ea into master Jun 17, 2024
33 of 34 checks passed
@Zales0123 Zales0123 deleted the custom-action-order-update branch June 17, 2024 11:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@loevgaard loevgaard loevgaard approved these changes

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Zales0123 @loevgaard