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

fix(events): get user events after login #390

Merged
merged 1 commit into from Sep 29, 2017

Conversation

lex111
Copy link
Member

@lex111 lex111 commented Sep 29, 2017

I have just noticed that after sign out the profile and when trying to log in, the events does not load, just a white screen, this PR fixes it.

@machour
Copy link
Member

machour commented Sep 29, 2017

@lex111 I noticed this behavior a few other times today, without signing in or out from the app.
Definitely something to keep an eye on before release

@lex111
Copy link
Member Author

lex111 commented Sep 29, 2017

@machour just after accepting this PR there will not be this error, because after login we do not have a user (auth user), we need to get it, and then it will come to the componentWillReceiveProps and it will load the events.

Copy link
Member

@housseindjirdeh housseindjirdeh left a comment

Choose a reason for hiding this comment

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

Beautiful, thank you so much @lex111 🎉

@housseindjirdeh housseindjirdeh merged commit 62559da into master Sep 29, 2017
@lex111 lex111 deleted the fix-load-user-events-after-logout branch September 29, 2017 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants