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

You can walk with riot shields #9403

Closed
Just-a-Unity-Dev opened this issue Jul 4, 2022 · 1 comment · Fixed by #9424
Closed

You can walk with riot shields #9403

Just-a-Unity-Dev opened this issue Jul 4, 2022 · 1 comment · Fixed by #9424
Labels
D3: Low Difficulty: Some codebase knowledge required. DB: Beginner Friendly Difficulty: Great for beginners. Unambiguous in scope, and explains how to achieve the result. P2: Raised Priority: Item has a raised priority, indicating it might get increased maintainer attention. T: Bugfix Type: Bugs and/or bugfixes

Comments

@Just-a-Unity-Dev
Copy link
Contributor

Description

You can apparently walk while having a raised riot shield

Reproduction

  1. Get a shield
  2. Raise the shield you have
  3. Sit down in a chair
  4. Unbuckle
  5. Presto! You are now solid!
@metalgearsloth
Copy link
Contributor

metalgearsloth commented Jul 4, 2022

Riot shields should probably subscribe to parent changes and cancel if it ever does. They should also check if you're buckled or contained and not allow it too.

@ShadowCommander ShadowCommander added T: Bugfix Type: Bugs and/or bugfixes P2: Raised Priority: Item has a raised priority, indicating it might get increased maintainer attention. D3: Low Difficulty: Some codebase knowledge required. DB: Beginner Friendly Difficulty: Great for beginners. Unambiguous in scope, and explains how to achieve the result. labels Jul 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
D3: Low Difficulty: Some codebase knowledge required. DB: Beginner Friendly Difficulty: Great for beginners. Unambiguous in scope, and explains how to achieve the result. P2: Raised Priority: Item has a raised priority, indicating it might get increased maintainer attention. T: Bugfix Type: Bugs and/or bugfixes
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants