Skip to content

UserCredentials and UserLogin with user_id#283

Merged
andreshndz merged 6 commits intomainfrom
user-credentials
Jun 20, 2022
Merged

UserCredentials and UserLogin with user_id#283
andreshndz merged 6 commits intomainfrom
user-credentials

Conversation

@andreshndz
Copy link
Copy Markdown
Contributor

@andreshndz andreshndz commented Jun 20, 2022

Now, we can send user_id when creating password and login

@andreshndz andreshndz self-assigned this Jun 20, 2022
@codecov
Copy link
Copy Markdown

codecov bot commented Jun 20, 2022

Codecov Report

Merging #283 (2567abc) into main (378542d) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main      #283   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           43        43           
  Lines         1005      1005           
=========================================
  Hits          1005      1005           
Flag Coverage Δ
unittests 100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
cuenca/resources/user_credentials.py 100.00% <100.00%> (ø)
cuenca/resources/user_logins.py 100.00% <100.00%> (ø)
cuenca/version.py 100.00% <100.00%> (ø)

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 378542d...2567abc. Read the comment docs.

@andreshndz andreshndz changed the title UserCredentials with user_id UserCredentials and UserLogin with user_id Jun 20, 2022
@andreshndz andreshndz requested a review from rogelioLpz June 20, 2022 19:20
@andreshndz andreshndz marked this pull request as ready for review June 20, 2022 22:24
@andreshndz andreshndz requested a review from alexviquez as a code owner June 20, 2022 22:24
alexviquez
alexviquez previously approved these changes Jun 20, 2022
Copy link
Copy Markdown
Contributor

@alexviquez alexviquez left a comment

Choose a reason for hiding this comment

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

versión

@andreshndz andreshndz merged commit 9b35427 into main Jun 20, 2022
@andreshndz andreshndz deleted the user-credentials branch June 20, 2022 23:07
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.

2 participants