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

Player MapView sending API #572

Open
aramperes opened this issue Oct 13, 2017 · 3 comments
Open

Player MapView sending API #572

aramperes opened this issue Oct 13, 2017 · 3 comments

Comments

@aramperes
Copy link
Member

@aramperes aramperes commented Oct 13, 2017

The API to send MapViews to a Player isn't implemented:

public void sendMap(MapView map) {
throw new UnsupportedOperationException("Not supported yet.");
}

This should send a packet (MapDataMessage) that sends the data inside of the MapView.

To check:

  • If the codec for encoding MapDataMessage is up-to-date with the documentation
  • If there are other parts of the MapView API that aren't implemented (not required in the scope of this issue)

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

@Pr0methean
Copy link
Contributor

@Pr0methean Pr0methean commented Dec 17, 2017

I will try my hand at this.

@aramperes aramperes closed this in 89f3731 Dec 17, 2017
@ghost ghost removed the help wanted label Dec 17, 2017
@mastercoms mastercoms reopened this Dec 17, 2017
aramperes pushed a commit that referenced this issue Dec 19, 2017
Related to #572
@aramperes
Copy link
Member Author

@aramperes aramperes commented Jan 18, 2018

What work remains in regard to this issue?

@Pr0methean
Copy link
Contributor

@Pr0methean Pr0methean commented Feb 5, 2018

I think just serializing/deserializing maps with NBT.

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

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.