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

Restore lock/lockall/locktrainer/release/releaseall in triggers #5976

Merged

Conversation

mrgriffin
Copy link
Collaborator

@mrgriffin mrgriffin commented Jan 8, 2025

#5033 caused some weirdness with lock* and release* commands in triggers. This PR should restore the behavior, but it needs some testing :)

Reported by AGSMGMaster64. Testing by HunarPG.

@AlexOn1ine
Copy link
Collaborator

how can this be tested?

@mrgriffin
Copy link
Collaborator Author

Annoyingly the answer is just "walk/run into triggers in-game and check they behave like in vanilla". Work's looking really busy atm and likely to bleed into my free time, but I'll report back if I find the time to do it :)

@mrgriffin mrgriffin force-pushed the rhh-rollback-mapscript branch from 5f67718 to a96a570 Compare January 13, 2025 11:31
@mrgriffin mrgriffin added the type: big feature A feature with big diffs and / or high impact / subjectivity / pervasiveness label Jan 14, 2025
@Pawkkie Pawkkie added the bugfix Bugfixes label Jan 14, 2025
@mrgriffin
Copy link
Collaborator Author

mrgriffin commented Jan 15, 2025

HunarPG found some problems that I need to fix:

@mrgriffin
Copy link
Collaborator Author

HunarPG found some problems that I need to fix:

Hunar signed off :)

@AsparagusEduardo AsparagusEduardo merged commit ed0c9a4 into rh-hideout:upcoming Jan 15, 2025
1 check passed
@hedara90 hedara90 added the category: overworld Pertains to out-of-battle mechanics label Feb 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix Bugfixes category: overworld Pertains to out-of-battle mechanics type: big feature A feature with big diffs and / or high impact / subjectivity / pervasiveness
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants