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

Allows to unlock all locks at once #1923

Merged
merged 1 commit into from
Jan 19, 2024
Merged

Allows to unlock all locks at once #1923

merged 1 commit into from
Jan 19, 2024

Conversation

sabieber
Copy link
Member

Description

Redefining the locals inside is a workaround for locals seemingly not being passed into a block.

Additional Notes

  • This PR fixes or works on following ticket(s): SIRI-928

Checklist

  • Code change has been tested and works locally
  • Code was formatted via IntelliJ and follows SonarLint & best practices
  • Patch Tasks: Is local execution of Patch Tasks necessary? If so, please also mark the PR with the tag.

@sabieber
Allows to unlock all locks at once
64017a2 
Redefining the locals inside is a workaround for locals seemingly not being passed into a block.

Fixes: SIRI-928
@sabieber sabieber added 🐛 Bugfix Contains only a small fix for an existing bug 👶🏻 Trivial Easy to review labels Jan 18, 2024
@idlira idlira merged commit 621954d into develop Jan 19, 2024
2 checks passed
@idlira idlira deleted the feature/sbi/unlock-all-button branch January 19, 2024 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@idlira idlira idlira approved these changes

@jakobvogel jakobvogel jakobvogel approved these changes

Assignees
No one assigned
Labels
🐛 Bugfix Contains only a small fix for an existing bug 👶🏻 Trivial Easy to review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sabieber @jakobvogel @idlira