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.

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

Fixed the fromPosition callback of some lua scripts that were registered by position #441

Merged
merged 1 commit into from
Jul 15, 2022

Conversation

dudantas
Copy link
Contributor

@dudantas dudantas commented Jul 15, 2022

Some scripts registered through position didn't work the fromPosition callback because it was lost between the two functions (with and without the callback item)

@dudantas dudantas changed the title Remove duplicated fireStepEvent and executeStep Fixed the fromPosition callback of some lua scripts that were registered by position Jul 15, 2022
@sonarcloud
Copy link

sonarcloud bot commented Jul 15, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@dudantas dudantas merged commit 723f378 into main Jul 15, 2022
@dudantas dudantas deleted the fix-position-scripts-register branch July 15, 2022 11:56
luan pushed a commit that referenced this pull request Jul 11, 2023
…by position (#441)

Some scripts registered through position didn't work the fromPosition callback because it was lost between the two functions (with and without the callback item)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant