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

PeerKit Additions #3

Merged
merged 12 commits into from Jan 22, 2015
Merged

PeerKit Additions #3

merged 12 commits into from Jan 22, 2015

Conversation

alexanderjarvis
Copy link
Contributor

Hello!

Thanks for writing PeerKit. I made some local additions, mostly delegate methods that weren't implemented, including sending and receiving resources.

I also added a way to customise the display name as it was just the device UUID by default with no way of changing it.

Also, a way to stop transceiving that disconnects the sessions and the ability to set a new Transceiver e.g. when the display name is changed by a user.

Thanks

@@ -11,41 +11,52 @@ import MultipeerConnectivity

// MARK: Type Aliases

public typealias PeerBlock = ((peerID: MCPeerID) -> Void)
public typealias PeerBlock = ((myPeerID:MCPeerID, peerID: MCPeerID) -> Void)
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you add a space here to match the rest of the code? myPeerID: MCPeerID

@jpsim
Copy link
Owner

jpsim commented Jan 22, 2015

This is fantastic, @alexanderjarvis! Thanks for the PR. I'll merge it once you address my two very minor comments.

@alexanderjarvis
Copy link
Contributor Author

@jpsim done!

jpsim added a commit that referenced this pull request Jan 22, 2015
@jpsim jpsim merged commit 92633e1 into jpsim:master Jan 22, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants