Skip to content
This repository was archived by the owner on Aug 29, 2018. It is now read-only.

Add port argument #69

Merged
merged 2 commits into from
Apr 18, 2017

Conversation

kostia-official
Copy link
Contributor

@kostia-official kostia-official commented Apr 14, 2017

I added port argument according to these discussions #66
Please check.

Closes feathersjs-ecosystem/docs#461

@daffl
Copy link
Member

daffl commented Apr 14, 2017

That looks great, thank you! Could you create another PR that adds the appropriate documentation below https://docs.feathersjs.com/api/socketio.html#appconfiguresocketiooptions--callback ? Then we can merge and release.

@kostia-official
Copy link
Contributor Author

@daffl Okay, I've added information about the port argument.
feathersjs-ecosystem/docs#527

@daffl daffl merged commit e5b43d6 into feathersjs-ecosystem:master Apr 18, 2017
@daffl
Copy link
Member

daffl commented Apr 18, 2017

Awesome, thank you! Released as v1.6.0 and documentation has been published as well.

@ekryski
Copy link
Member

ekryski commented Apr 18, 2017

Nice @kozzztya! Thanks for all the great PRs! 😄

@kostia-official
Copy link
Contributor Author

You're welcome.

# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants