Skip to content

feat: [OCISDEV-518] Added the graph endpoint alias#11871

Merged
2403905 merged 2 commits intomasterfrom
feat/OCISDEV-518
Dec 22, 2025
Merged

feat: [OCISDEV-518] Added the graph endpoint alias#11871
2403905 merged 2 commits intomasterfrom
feat/OCISDEV-518

Conversation

@2403905
Copy link
Copy Markdown
Contributor

@2403905 2403905 commented Dec 15, 2025

Description

The OCIS backend exposes the graph api v1 and v1beta1. The /graph/v1/drives/{id} endpoint is available only in v1 and returns the role aliases against the role ID in a v1beta1. Then, the web converts the role aliases to ID via hardcoded mapping. web/packages/web-client/src/graph/drives/drives.ts at 21b74354803b09ca6abc1175a0419e6390e6d466 · owncloud/web

We should implement the v1beta1 for /graph/{api_version}/drives/{id}

Related Issue

Motivation and Context

How Has This Been Tested?

  • test environment:
  • test case 1:
  • test case 2:
  • ...

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:

@2403905 2403905 requested a review from kobergj December 16, 2025 08:39
@2403905 2403905 force-pushed the feat/OCISDEV-518 branch 2 times, most recently from a8709e8 to 67fdab9 Compare December 16, 2025 12:38
@2403905 2403905 requested a review from mmattel December 16, 2025 17:31
@sonarqubecloud
Copy link
Copy Markdown

@2403905 2403905 merged commit fb95b57 into master Dec 22, 2025
5 checks passed
@2403905 2403905 deleted the feat/OCISDEV-518 branch December 22, 2025 12:01
ownclouders pushed a commit that referenced this pull request Dec 22, 2025
* feat: [OCISDEV-518] Added the graph endpointis alias

* Update changelog/unreleased/enhancement-add-graph-endpoint.md

Co-authored-by: Martin <github@diemattels.at>

---------

Co-authored-by: Martin <github@diemattels.at>
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.

3 participants