-
-
Notifications
You must be signed in to change notification settings - Fork 883
Fix #4614: Resolve profile update failure when modifying First Name, Last Name, or Affiliation fields. #4629
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
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #4629 +/- ##
==========================================
- Coverage 72.93% 69.63% -3.30%
==========================================
Files 83 21 -62
Lines 5368 3610 -1758
==========================================
- Hits 3915 2514 -1401
+ Misses 1453 1096 -357
... and 65 files with indirect coverage changes
... and 65 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
Thanks @Amarta113 for the fix! |
Hi @RishabhJain2018, thank you for reviewing and approving the PR. I noticed that merging is currently blocked due to authorization. Could you please advise on how to proceed? |
:) You can't merge the PR. I'll merge it. |
Hey @RishabhJain2018 is it ready to merge or do i need any changes? |
This PR fixes #4614
Changes Details:
Corrected the handling of the urlFields during profile updates.
Ensured that updates to First Name, Last Name, and Affiliation fields work correctly without triggering an error.
Verified that other fields like username and social links continue to update properly.
Testing Done:
Manually tested updating each of the affected fields individually and together.
Confirmed that the error no longer appears and that the profile updates successfully.
Output Picture: