-
Notifications
You must be signed in to change notification settings - Fork 48
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
Implementation presence functionality #13
Conversation
Thanks Justin. I've been a bit time poor so am just getting around to reviewing this. I've also gotten Travis CI setup (rather than codeship). Would be good to rebase your work and have it run the tests on CI Also FYI, have updated testing so it can now just run via |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a really welcome addition. I've provided some suggestions to consider.
Co-Authored-By: j12n <j12n@users.noreply.github.com>
Co-Authored-By: j12n <j12n@users.noreply.github.com>
Hey Martin, thanks for the great feedback. I took a quick pass and addressed. |
Great. One last thing. Can you add an entry to the Changelog? (Next release with your stuff will be 0.2.0). It'll be the first Changelog entry, but would be nice to add some rigour around documentation :) |
Done! I added placeholders for versions 0.1.2 and 0.1.3 but didn't add the details. |
Awesome |
Ok, release published https://pub.dartlang.org/packages/phoenix_wings/versions/0.2.0 Thanks so much @j12n |
Issue #1
I've taken a crack at implementing the phoenix presence functionality. Both core logic and tests are a direct port of the javascript code. All tests are passing but I haven't tested it in a real flutter app yet.
~Justin