-
Notifications
You must be signed in to change notification settings - Fork 187
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
[test-only] ApiTest. checking the owner of the project space #5342
Conversation
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 can we add this to the expected failures and merge it? I want to test it in #5341 which should make it pass. |
done |
thanks, i cherry pick it in #5341 to see if it passes, give me one more ci run ;) |
@ScharfViktor i had a look, the space owner is still the spaceId (project space only). Therefor the test still fails (the graph drives endpoint still uses the spaceId as owner). @butonic FYI |
For me it's okay because it doesn't break any business logic and everything still looks good on the UI |
related #4419