Adding destroyBarricade override to BarricadeManager. #3744
Jdance-Media
started this conversation in
Ideas
Replies: 2 comments 1 reply
-
Ideally yes in the future plugins should not need to track barricade region at all, it should be an internal implementation detail. |
Beta Was this translation helpful? Give feedback.
0 replies
-
You don't know this game very well. |
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Currently,
destroyBarricade
finds the region based on the x and y bytes and destroys the specified barricade within that region. This is helpful when you don't have theBarricadeRegion
already, but it seems redundant in the many cases where you already have the region. It would be helpful if there was aBarricadeManager.destroyBarricade(BarricadeRegion region, BarricadeDrop drop)
method.(Pssst, we don't have access to
SendDestroyBarricade
, so that's mainly why).Beta Was this translation helpful? Give feedback.
All reactions