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

calls loginWithToken callback also on success #3690

Merged

Conversation

thomasspiesser
Copy link
Contributor

@RocketChat/core

Currently, the loginWithToken callback is only invoked on error from the login. Thus, this.reaciveIframeUrl.set() on this line

is never reached in case of success. Seems like it should though.

…iframe-loginWithToken-callback

* 'develop' of github.com:thomasspiesser/Rocket.Chat:
  Add missing translations
  LingoHub Update 🚀 (RocketChat#3684)
  Update FEATURES.md
  Create FEATURES.md
  Add missing roles verification to direct messages (RocketChat#3672)
@thomasspiesser thomasspiesser changed the title calls callback also on success calls loginWithToken callback also on success Jul 6, 2016
@rodrigok
Copy link
Member

rodrigok commented Jul 6, 2016

Awesome, thanks

@rodrigok rodrigok merged commit a79af9d into RocketChat:develop Jul 6, 2016
thomasspiesser added a commit to thomasspiesser/Rocket.Chat that referenced this pull request Jul 6, 2016
…filter-notify-all

* 'develop' of github.com:thomasspiesser/Rocket.Chat:
  add editable channel descriptions (RocketChat#3705)
  Fix sandstorm WebRTC (RocketChat#3675)
  Subdir images  (RocketChat#3695)
  calls callback also on success (RocketChat#3690)
  don't consider invalid commands to be messages (RocketChat#3698)
  Add hide avatars setting to user preferences
@marceloschmidt marceloschmidt added this to the 0.36.0 milestone Sep 5, 2016
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.

3 participants