Skip to content

[PM-38928] - Add name, email to org user update#7871

Open
jrmccannon wants to merge 3 commits into
mainfrom
jmccannon/ac/pm-38928-add-name-email-edit-member
Open

[PM-38928] - Add name, email to org user update#7871
jrmccannon wants to merge 3 commits into
mainfrom
jmccannon/ac/pm-38928-add-name-email-edit-member

Conversation

@jrmccannon

@jrmccannon jrmccannon commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

🎟️ Tracking

PM-38928

📔 Objective

Adds optional parameters to the PUT org user endpoint. These will be used in future work.

@jrmccannon jrmccannon requested a review from a team as a code owner June 25, 2026 16:30
@jrmccannon jrmccannon requested a review from eliykat June 25, 2026 16:30
@github-actions

Copy link
Copy Markdown
Contributor

Claude Code is reviewing this pull request...

If this comment does not update with results, check the Actions log.

@jrmccannon jrmccannon added the t:feature Change Type - Feature Development label Jun 25, 2026
eliykat
eliykat previously approved these changes Jun 25, 2026

@eliykat eliykat left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Are these nullable? If so, please mark them as such. Otherwise lgtm.

@sonarqubecloud

Copy link
Copy Markdown

@codecov

codecov Bot commented Jun 26, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 71.42857% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 61.21%. Comparing base (f5ec38e) to head (5f90094).

Files with missing lines Patch % Lines
src/Core/Utilities/StrictEmailAddressAttribute.cs 60.00% 0 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7871      +/-   ##
==========================================
- Coverage   61.21%   61.21%   -0.01%     
==========================================
  Files        2219     2219              
  Lines       98052    98059       +7     
  Branches     8846     8848       +2     
==========================================
+ Hits        60020    60024       +4     
- Misses      35917    35918       +1     
- Partials     2115     2117       +2     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t:feature Change Type - Feature Development

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants