Skip to content

Require given and fullname from SCIM - #33336

Merged
MagnusHJensen merged 2 commits into
mainfrom
require-given-and-fullname-from-scim
Sep 23, 2025
Merged

Require given and fullname from SCIM#33336
MagnusHJensen merged 2 commits into
mainfrom
require-given-and-fullname-from-scim

Conversation

@MagnusHJensen

Copy link
Copy Markdown
Member

Related issue: Followup fix for #30888

See #30888 (comment)

Needs to be cherry-picked into 4.74

Checklist for submitter

If some of the following don't apply, delete the relevant line.

Testing

  • Added/updated automated tests
  • QA'd all new/changed functionality manually

@codecov

codecov Bot commented Sep 23, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 64.08%. Comparing base (6ccf1a0) to head (1b75443).
⚠️ Report is 29 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #33336      +/-   ##
==========================================
+ Coverage   63.96%   64.08%   +0.12%     
==========================================
  Files        2054     2058       +4     
  Lines      202785   205014    +2229     
  Branches     6719     6719              
==========================================
+ Hits       129712   131384    +1672     
- Misses      62865    63259     +394     
- Partials    10208    10371     +163     
Flag Coverage Δ
backend 65.19% <100.00%> (+0.12%) ⬆️

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.

🚀 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.

@MagnusHJensen
MagnusHJensen merged commit 4e56d53 into main Sep 23, 2025
39 checks passed
@MagnusHJensen
MagnusHJensen deleted the require-given-and-fullname-from-scim branch September 23, 2025 14:09
MagnusHJensen added a commit that referenced this pull request Sep 23, 2025
<!-- Add the related story/sub-task/bug number, like Resolves #123, or
remove if NA -->
**Related issue:** Followup fix for #30888

See
#30888 (comment)

Needs to be cherry-picked into 4.74

# Checklist for submitter

If some of the following don't apply, delete the relevant line.


## Testing

- [x] Added/updated automated tests
- [x] QA'd all new/changed functionality manually
MagnusHJensen added a commit that referenced this pull request Sep 23, 2025
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