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

[SDK-4665] Support initiate_login_uri property being an empty string for PATCH /api/v2/clients/:id requests. #732

Merged
merged 3 commits into from
Oct 16, 2023

Conversation

evansims
Copy link
Member

Changes

This PR adds support for initiate_login_uri to be supplied as an empty string to the /api/v2/clients/:id endpoint for PATCH requests.

References

Resolves #731

Testing

This PR adds a regression testing for the change.

Contributor Checklist

@evansims evansims linked an issue Oct 12, 2023 that may be closed by this pull request
7 tasks
@evansims evansims temporarily deployed to internal October 12, 2023 19:40 — with GitHub Actions Inactive
@evansims evansims temporarily deployed to internal October 12, 2023 19:40 — with GitHub Actions Inactive
@evansims evansims temporarily deployed to internal October 12, 2023 19:40 — with GitHub Actions Inactive
@evansims evansims added the Scope: Bug Addressing unexpected problems or unintended behavior. label Oct 12, 2023
@evansims evansims temporarily deployed to internal October 12, 2023 19:47 — with GitHub Actions Inactive
@evansims evansims temporarily deployed to internal October 12, 2023 19:47 — with GitHub Actions Inactive
@evansims evansims temporarily deployed to internal October 12, 2023 19:47 — with GitHub Actions Inactive
@evansims evansims marked this pull request as ready for review October 12, 2023 19:49
@evansims evansims requested a review from a team as a code owner October 12, 2023 19:49
@codecov-commenter
Copy link

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (9e0c5ed) 100.00% compared to head (5dfaffe) 100.00%.

Additional details and impacted files
@@             Coverage Diff             @@
##                main      #732   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
- Complexity      1327      1330    +3     
===========================================
  Files             62        62           
  Lines           4613      4615    +2     
===========================================
+ Hits            4613      4615    +2     
Flag Coverage Δ
unittestsvalidate 100.00% <100.00%> (ø)

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

Files Coverage Δ
src/API/Management/Clients.php 100.00% <100.00%> (ø)

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

@evansims evansims added this pull request to the merge queue Oct 16, 2023
Merged via the queue into main with commit 3474acd Oct 16, 2023
22 checks passed
@evansims evansims deleted the 731-cannot-empty-initiate_login_uri-with-api branch October 16, 2023 01:43
@evansims evansims mentioned this pull request Oct 18, 2023
github-merge-queue bot pushed a commit that referenced this pull request Oct 18, 2023
### Added

- Support `initiate_login_uri` property for `PATCH` requests to the
`/api/v2/clients/:id` Management API endpoint.
[#732](#732)

### Fixed

- Token generator typo in claim header.
[#729](#729)

### Changed

- `composer.json` now identifies `mbstring` as a required extension.
[#730](#730)
- PHP 8.3 nightlies have been added to the continuous integration
testing matrix. [#733](#733)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scope: Bug Addressing unexpected problems or unintended behavior.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cannot empty initiate_login_uri with API
3 participants