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

just returned updated user details after updating user #3660

Merged
merged 1 commit into from
Oct 15, 2024

Conversation

Baalmart
Copy link
Contributor

@Baalmart Baalmart commented Oct 15, 2024

Description

just returned updated user details after updating user

Changes Made

  • just returned updated user details after updating user

Testing

  • Tested locally
  • Tested against staging environment
  • Relevant tests passed: [List test names]

Affected Services

  • Which services were modified:
    • Auth Service

Endpoints Ready for Testing

  • New endpoints ready for testing:
    • Update User

API Documentation Updated?

  • Yes, API documentation was updated
  • No, API documentation does not need updating

Copy link

codecov bot commented Oct 15, 2024

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 27.04%. Comparing base (8f6a455) to head (fde95e8).
Report is 3 commits behind head on staging.

Files with missing lines Patch % Lines
src/auth-service/models/User.js 0.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           staging    #3660      +/-   ##
===========================================
- Coverage    27.04%   27.04%   -0.01%     
===========================================
  Files          146      146              
  Lines        21339    21340       +1     
  Branches       274      274              
===========================================
  Hits          5772     5772              
- Misses       15567    15568       +1     
Files with missing lines Coverage Δ
src/auth-service/models/User.js 14.08% <0.00%> (-0.07%) ⬇️

Copy link
Contributor

Auth-service changes in this PR available for preview here

@Baalmart Baalmart merged commit 839fc94 into staging Oct 15, 2024
49 of 50 checks passed
@Baalmart Baalmart deleted the hf-update-user branch October 15, 2024 18:32
@Baalmart Baalmart mentioned this pull request Oct 15, 2024
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant