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

Fix possible ender pearl issue with being ale to insta teleport #27

Open
rourke750 opened this issue May 2, 2015 · 7 comments
Open

Comments

@rourke750
Copy link

http://www.reddit.com/r/Civcraft/comments/34k2nz/will_enderpearl_teleportion_machineteleporter_be/cqvl6ud

@ttk2
Copy link

ttk2 commented May 2, 2015

yup, thanks for making the issue, should we just check pearl events for the
distance thrown? I wonder if there is any way to limit our check overhead
to just this one situation.

On Fri, May 1, 2015 at 11:21 PM rourke750 [email protected] wrote:

http://www.reddit.com/r/Civcraft/comments/34k2nz/will_enderpearl_teleportion_machineteleporter_be/cqvl6ud


Reply to this email directly or view it on GitHub
#27.

@rourke750
Copy link
Author

Every entity has an id. I'm going to schedule a timer and if it shows up
I'm just going to delete it.
On May 2, 2015 12:27 AM, "ttk2" [email protected] wrote:

yup, thanks for making the issue, should we just check pearl events for the
distance thrown? I wonder if there is any way to limit our check overhead
to just this one situation.

On Fri, May 1, 2015 at 11:21 PM rourke750 [email protected]
wrote:

http://www.reddit.com/r/Civcraft/comments/34k2nz/will_enderpearl_teleportion_machineteleporter_be/cqvl6ud


Reply to this email directly or view it on GitHub
#27.


Reply to this email directly or view it on GitHub
#27 (comment).

@rourke750
Copy link
Author

possibly fixed 031c832

@ttk2
Copy link

ttk2 commented May 24, 2015

this should be resolved right?

@Xavter
Copy link

Xavter commented May 24, 2015

Me and sanwi will build a pearl machine later, we can double check it then

@suirad
Copy link

suirad commented Sep 4, 2015

Was this tested?

@suirad
Copy link

suirad commented Sep 30, 2015

@Xavter did you happen to test this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants