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

Return promise in signIn #5021

Merged
merged 2 commits into from
Mar 26, 2020
Merged

Return promise in signIn #5021

merged 2 commits into from
Mar 26, 2020

Conversation

janoist1
Copy link
Contributor

@janoist1 janoist1 commented Mar 2, 2020

Issue #, if available: SignIn.signIn does not return the Promise got from Auth.signIn. This is not consistent with the React Web version which does return it.

Description of changes: Return promise.

@janoist1 janoist1 requested a review from Ashish-Nanda as a code owner March 2, 2020 17:41
@janoist1
Copy link
Contributor Author

Please someone at least react to this...

@codecov
Copy link

codecov bot commented Mar 25, 2020

Codecov Report

Merging #5021 into master will not change coverage by %.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #5021   +/-   ##
=======================================
  Coverage   73.21%   73.21%           
=======================================
  Files         193      193           
  Lines       11441    11441           
  Branches     2218     2302   +84     
=======================================
  Hits         8377     8377           
+ Misses       2916     2900   -16     
- Partials      148      164   +16     
Impacted Files Coverage Δ
packages/auth/src/OAuth/OAuth.ts 48.12% <0.00%> (ø)
packages/datastore/src/sync/outbox.ts 25.00% <0.00%> (ø)
packages/datastore/src/storage/storage.ts 67.59% <0.00%> (ø)
packages/core/src/OAuthHelper/GoogleOAuth.ts 31.25% <0.00%> (ø)
packages/xr/src/Providers/SumerianProvider.ts 47.55% <0.00%> (ø)
packages/core/src/OAuthHelper/FacebookOAuth.ts 34.09% <0.00%> (ø)
packages/datastore/src/sync/processors/sync.ts 18.05% <0.00%> (ø)
...ackages/datastore/src/storage/adapter/indexeddb.ts 73.26% <0.00%> (ø)
...ackages/pubsub/src/Providers/AWSAppSyncProvider.ts 66.66% <0.00%> (ø)
...ages/datastore/src/sync/processors/subscription.ts 43.91% <0.00%> (ø)
... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4cf7b72...7ee61db. Read the comment docs.

@janoist1
Copy link
Contributor Author

No one can review a single line commit in 3 weeks? 😷

@manueliglesias
Copy link
Contributor

Hi @janoist1

We are reviewing and testing your PR today, thank you for the contribution!

Copy link
Contributor

@Ashish-Nanda Ashish-Nanda left a comment

Choose a reason for hiding this comment

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

@janoist1 Thank you for your contribution!

I've tested and approved your PR, and the changes will be available in the next release.

@Ashish-Nanda Ashish-Nanda merged commit cb902b0 into aws-amplify:master Mar 26, 2020
@github-actions
Copy link

This pull request has been automatically locked since there hasn't been any recent activity after it was closed. Please open a new issue for related bugs.

Looking for a help forum? We recommend joining the Amplify Community Discord server *-help channels or Discussions for those types of questions.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants