Skip to content
This repository has been archived by the owner on Nov 11, 2021. It is now read-only.

Feature/accounts loader #225

Merged
merged 9 commits into from Nov 22, 2018
Merged

Feature/accounts loader #225

merged 9 commits into from Nov 22, 2018

Conversation

vladimirvolek
Copy link
Contributor

fix #66

@vladimirvolek vladimirvolek self-assigned this Oct 31, 2018
@vladimirvolek vladimirvolek added the request review Need someone to review a solution for an issue label Oct 31, 2018
@vladimirvolek
Copy link
Contributor Author

@szymonlesisz maybe we can also add "Authenticating device" to loader?

szymonlesisz
szymonlesisz approved these changes Nov 22, 2018
View reviewed changes
Copy link
Contributor

@szymonlesisz szymonlesisz left a comment

Choose a reason for hiding this comment

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

I've added one more (default) case into SelectedAccount loader

@szymonlesisz szymonlesisz merged commit 556b2be into master Nov 22, 2018
@szymonlesisz szymonlesisz deleted the feature/accounts-loader branch November 22, 2018 13:19
@szymonlesisz szymonlesisz mentioned this pull request Nov 26, 2018
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@szymonlesisz szymonlesisz szymonlesisz approved these changes

Assignees

@vladimirvolek vladimirvolek

Labels
request review Need someone to review a solution for an issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add accounts loader remove notification
2 participants
@vladimirvolek @szymonlesisz