Skip to content

[Application] Merge process files on both parallel and non-parallel process#4523

Merged
TomasVotruba merged 11 commits intomainfrom
merge-process-files
Jul 19, 2023
Merged

[Application] Merge process files on both parallel and non-parallel process#4523
TomasVotruba merged 11 commits intomainfrom
merge-process-files

Conversation

@samsonasik
Copy link
Copy Markdown
Member

@TomasVotruba @staabm continue of PRs:

this PR merge process between parallel and non-parallel to avoid miss logic applied in the future.

@samsonasik samsonasik requested a review from TomasVotruba as a code owner July 15, 2023 13:30
@samsonasik samsonasik marked this pull request as draft July 15, 2023 13:57
ReactCommand::ACTION => Action::RESULT,
self::RESULT => [
Bridge::SYSTEM_ERRORS => [$systemErrors],
Bridge::SYSTEM_ERRORS => [$systemError],
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@samsonasik samsonasik marked this pull request as ready for review July 19, 2023 04:28
@samsonasik
Copy link
Copy Markdown
Member Author

@TomasVotruba Ready for review 👍 . check-active-class CI notice seems related with PR:

@samsonasik samsonasik force-pushed the merge-process-files branch from 09e8215 to a1ce1b5 Compare July 19, 2023 10:43
@samsonasik
Copy link
Copy Markdown
Member Author

Rebased 👍

@samsonasik
Copy link
Copy Markdown
Member Author

All checks have passed 🎉 @TomasVotruba it is ready for review.

@TomasVotruba
Copy link
Copy Markdown
Member

TomasVotruba commented Jul 19, 2023

Let's give this a try :) thanks 👍

@TomasVotruba TomasVotruba merged commit 972d760 into main Jul 19, 2023
@TomasVotruba TomasVotruba deleted the merge-process-files branch July 19, 2023 10:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants