search

Civcraft / JukeAlert

Do not open issues here; open them on the maintained fork @ DevotedMC
https://github.com/DevotedMC/JukeAlert
BSD 3-Clause "New" or "Revised" License
5 stars 15 forks source link

jalookup should assume the player's current world #10

Closed erocs closed 9 years ago

erocs commented 9 years ago

jalookup's 4th parameter is the world name for the juke. Right now it's hardcoded to 'world' but it should instead use the player's current world.

rourke750 commented 9 years ago

Doing /jalookup x y z world_nether works? Just tried it.

On Sun, Dec 28, 2014 at 4:33 PM, erocs notifications@github.com wrote:

jalookup's 4th parameter is the world name for the juke. Right now it's hardcoded to 'world' but it should instead use the player's current world.

— Reply to this email directly or view it on GitHub https://github.com/Civcraft/JukeAlert/issues/10.

erocs commented 9 years ago

Yes, and if someone is in the nether they should just be able to /jalookup x y z and get their nether snitch at that location.

On Sun, Dec 28, 2014 at 1:35 PM, rourke750 notifications@github.com wrote:

Doing /jalookup x y z world_nether works? Just tried it.

On Sun, Dec 28, 2014 at 4:33 PM, erocs notifications@github.com wrote:

jalookup's 4th parameter is the world name for the juke. Right now it's hardcoded to 'world' but it should instead use the player's current world.

— Reply to this email directly or view it on GitHub https://github.com/Civcraft/JukeAlert/issues/10.

— Reply to this email directly or view it on GitHub https://github.com/Civcraft/JukeAlert/issues/10#issuecomment-68220340.

rourke750 commented 9 years ago

https://github.com/Civcraft/JukeAlert/commit/bc2609c2fb4fc8d71d817ec12e222ee4336bec21