Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update oc_rlvsuite.lsl #947

Merged
merged 1 commit into from
May 17, 2023
Merged

Conversation

Medea-Destiny
Copy link
Collaborator

Stray restriction sets sit instead of sittp (instructions say it stops you sitting on things unless you're standing right next to them, which is sittp, it currently sets sit). This fix changes the stray restriction values for sittp rather than sit.

Untested in world so far, someone just brought it up in the group so I put in the fix, so needs checking before approving the pull, but there shouldn't be an issue unless I've typod or done something dumb!

@Medea-Destiny Medea-Destiny added Do Not Merge (yet) One reason or another for not merging 8.3 labels Apr 12, 2023
Copy link
Collaborator

@Pingout Pingout left a comment

Choose a reason for hiding this comment

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

This script works as intended. The sit restriction has been replaced with a sittp restriction when using the Stray Preset.
With Strict set, wearer is allowed to sit on objects that are closer than 1 meter, and restricted from sitting on objects at a distance greater than 1 meter

@SilkieSabra
Copy link
Contributor

This looks tested inworld, approved and ready to merge. Any reason there is a Do Not Merge yet label on it still?

@Medea-Destiny Medea-Destiny removed the Do Not Merge (yet) One reason or another for not merging label May 4, 2023
@Medea-Destiny
Copy link
Collaborator Author

Nope, do not merge was just there pending in world testing!

@SilkieSabra SilkieSabra merged commit 8f4ad07 into 8.3_Features-branch May 17, 2023
@Medea-Destiny Medea-Destiny deleted the Medea-Destiny-rlvsuite-1 branch October 7, 2024 21:39
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants