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

fix(login): revert provider user ID format #112

Merged
merged 1 commit into from
Mar 1, 2024
Merged

fix(login): revert provider user ID format #112

merged 1 commit into from
Mar 1, 2024

Conversation

frgfm
Copy link
Member

@frgfm frgfm commented Mar 1, 2024

This PR fixes a typo in the format used for provider_user_id, which was causing a crash on the login route.

@frgfm frgfm added the type: fix label Mar 1, 2024
@frgfm frgfm added this to the 0.1.0 milestone Mar 1, 2024
@frgfm frgfm self-assigned this Mar 1, 2024
Copy link

codecov bot commented Mar 1, 2024

Codecov Report

Attention: Patch coverage is 50.00000% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 83.35%. Comparing base (4f48666) to head (3724a23).
Report is 1 commits behind head on main.

Files Patch % Lines
src/app/api/api_v1/endpoints/login.py 50.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #112   +/-   ##
=======================================
  Coverage   83.35%   83.35%           
=======================================
  Files          29       29           
  Lines         883      883           
=======================================
  Hits          736      736           
  Misses        147      147           
Flag Coverage Δ
unittests 83.35% <50.00%> (ø)

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@frgfm frgfm merged commit 0542e2d into main Mar 1, 2024
17 of 18 checks passed
@frgfm frgfm deleted the login-fix branch March 1, 2024 12:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant