Skip to content
This repository was archived by the owner on Jun 30, 2022. It is now read-only.

feat(users): use setUpUserProfile if there is a chance it is a new user #136

Merged
satello merged 1 commit into from
May 10, 2018
Merged

feat(users): use setUpUserProfile if there is a chance it is a new user #136

satello merged 1 commit into from
May 10, 2018

Conversation

@satello
Copy link
Contributor

@satello satello commented May 10, 2018

  • Use setUpUserProfile for any function that interacts with the userProfile which could be from a new user.

@coveralls
Copy link

coveralls commented May 10, 2018

Pull Request Test Coverage Report for Build 655

  • 0 of 10 (0.0%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.1%) to 43.372%
Changes Missing Coverage Covered Lines Changed/Added Lines %
src/resources/Disputes.js 0 1 0.0%
src/resources/Notifications.js 0 2 0.0%
src/utils/StoreProviderWrapper.js 0 7 0.0%
Totals Coverage Status
Change from base Build 652: -0.1%
Covered Lines: 421
Relevant Lines: 847
💛 - Coveralls

@satello satello merged commit 7734223 into develop May 10, 2018
@satello satello deleted the feat/handle_new_users_store branch May 10, 2018 16:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@satello @coveralls