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.

Sign up for GitHub

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

Jump to bottom

[test-only] add federated users to space #10166

Merged
merged 1 commit into from
Oct 8, 2024
Merged

[test-only] add federated users to space #10166

merged 1 commit into from
Oct 8, 2024

Conversation

ScharfViktor
Copy link
Contributor

@ScharfViktor ScharfViktor commented Sep 25, 2024 •
edited by saw-jan
Loading

related #10060
#10051

  • user tries to send the following space share invitation using permissions endpoint to federated user using the Graph API Actual: 200 Expected: 403
  • user tries to send the following space share invitation using root endpoint to federated user using the Graph API - Actual: 400

cc @2403905

@update-docs Update Docs
Copy link

update-docs bot commented Sep 25, 2024

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@ScharfViktor ScharfViktor mentioned this pull request Oct 7, 2024
Closed
@ScharfViktor ScharfViktor force-pushed the testForAddingFederatedUserstoSpace branch from 7eba6e5 to 23bc315 Compare October 7, 2024 13:57
saw-jan
saw-jan reviewed Oct 8, 2024
View reviewed changes
tests/acceptance/features/apiOcm/share.feature Outdated Show resolved Hide resolved
tests/acceptance/features/apiOcm/share.feature Outdated Show resolved Hide resolved
tests/acceptance/features/apiOcm/share.feature Outdated Show resolved Hide resolved
tests/acceptance/features/apiOcm/share.feature Outdated Show resolved Hide resolved
This was referenced Oct 8, 2024
Closed
Open
@ScharfViktor ScharfViktor force-pushed the testForAddingFederatedUserstoSpace branch from 23bc315 to 43479e9 Compare October 8, 2024 06:28
@ScharfViktor ScharfViktor requested a review from saw-jan October 8, 2024 06:28
@ScharfViktor ScharfViktor enabled auto-merge October 8, 2024 06:28
@ScharfViktor ScharfViktor force-pushed the testForAddingFederatedUserstoSpace branch from 43479e9 to 6de98f5 Compare October 8, 2024 06:55
@sonarqubecloud SonarQubeCloud
Copy link

sonarqubecloud bot commented Oct 8, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

saw-jan
saw-jan approved these changes Oct 8, 2024
View reviewed changes
Copy link
Member

@saw-jan saw-jan left a comment

Choose a reason for hiding this comment

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

👍

@ScharfViktor ScharfViktor merged commit 7310883 into master Oct 8, 2024
2 checks passed
@saw-jan saw-jan deleted the testForAddingFederatedUserstoSpace branch October 8, 2024 07:57
ownclouders pushed a commit that referenced this pull request Oct 8, 2024
@ScharfViktor
Merge pull request #10166 from owncloud/testForAddingFederatedUsersto…
01d3ea1 
…Space

[test-only] add federated users to space
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@saw-jan saw-jan saw-jan approved these changes

@2403905 2403905 Awaiting requested review from 2403905

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ScharfViktor @saw-jan