Skip to content

Corrected order of execution of code in step - #288

Merged
ejunprung merged 1 commit into
devfrom
se_print_debug_before_executing_action
Apr 2, 2021
Merged

Corrected order of execution of code in step#288
ejunprung merged 1 commit into
devfrom
se_print_debug_before_executing_action

Conversation

@SaharEs

@SaharEs SaharEs commented Mar 26, 2021 •
edited
Loading

Copy link
Copy Markdown
Contributor

Closes #284

Here is the new order:

  1. computes the action
  2. prints the debug
  3. executes the actions

@SaharEs
corrected order of execution
58a9650 
computes the action, prints the debug, then executes the actions
@SaharEs
SaharEs requested a review from ejunprung March 26, 2021 15:48
@ejunprung

ejunprung commented Mar 26, 2021

Copy link
Copy Markdown
Contributor

Thanks, I'll test this on Monday.

ejunprung
ejunprung approved these changes Apr 2, 2021
View reviewed changes

@ejunprung ejunprung left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Tested and works

@ejunprung
ejunprung merged commit 6809ef3 into dev Apr 2, 2021
@ejunprung
ejunprung deleted the se_print_debug_before_executing_action branch April 2, 2021 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ejunprung ejunprung ejunprung approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Debug mode output bug

2 participants

@SaharEs @ejunprung