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

Leash for pets #10256

Closed
kevingranade opened this issue Nov 28, 2014 · 4 comments · Fixed by #50890
Closed

Leash for pets #10256

kevingranade opened this issue Nov 28, 2014 · 4 comments · Fixed by #50890
Labels
<Enhancement / Feature> New features, or enhancements on existing (S2 - Confirmed) Bug that's been confirmed to exist

Comments

@kevingranade
Copy link
Member

Brought up by Tnx on IRC.
The feature would allow you to craft a leash (or possibly just use a rope directly) which you can attach to a pet (zombie or dog, doesn't matter). When attached, the leash would prevent the pet from moving more than n squares away from the player, and if the player moves more than that distance away from the pet, it would drag the pet.

Robustness:
Player should drop the leash if vehicle motion or forced movement moves them out of range.
Pet should be able to navigate stairs, possibly just by arriving at the bottom of the stairs at the same time as the player.

Optional features:
Strength check for pet dragging player or vice versa.
Have the pet navigate stairs properly instead of being teleported.
Allow multiple leashed pets, which ties (ha) back into the strength check thing.
Offensive use of a leash, such as a lasso type effect allowing use on hostile creatures.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@kevingranade kevingranade added the <Enhancement / Feature> New features, or enhancements on existing label Nov 28, 2014
@KA101
Copy link
Contributor

KA101 commented Nov 28, 2014

Leash wouldn't lasso. That's a rope thing. ;-)

@kevingranade kevingranade added the (S2 - Confirmed) Bug that's been confirmed to exist label Dec 8, 2014
@mugling
Copy link
Contributor

mugling commented Mar 9, 2016

Does the pet carrier provide enough of a solution?

@kevingranade
Copy link
Member Author

kevingranade commented Mar 9, 2016 via email

@illi-kun
Copy link
Contributor

illi-kun commented Nov 6, 2016

This issue was closed as it appears inactive.

Reducing open issues to those which are (or will) be actively worked upon helps us focus our efforts. This issue has not been deleted - it still appears in searches and if it contains relevant information you are encouraged to continue to link to it.

If this issue was a bug

It should be reopened if it can be reproduced in the current build. You can obtain the most recent copy here. Please check there is not a more recent report of this bug before doing so. If no more recent report exists you should continue the discussion in this issue.

If this was a feature request

If the consensus was that the idea was good you could consider submitting an implementation via a PR. If you want to comment further please do so here as opposed to opening a new issue. Before posting check nobody has already made the same point and consider whether your comments are likely to lead to an implementation. If you have doubts about either consider instead voting for the issue

If you want to work on this issue

Then either assign it to yourself or if you are unable to do so claim it via adding a comment. Please don't assign others or make a general request for action.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Enhancement / Feature> New features, or enhancements on existing (S2 - Confirmed) Bug that's been confirmed to exist
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants