Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Adapt to OCM spec for remote shares#57153

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
enriquepablo wants to merge8 commits intonextcloud:master
base:master
Choose a base branch
Loading
fromenriquepablo:master

Conversation

@enriquepablo
Copy link

@enriquepabloenriquepablo commentedDec 17, 2025•
edited
Loading

Summary

This is a preliminary PR for discussion. If this goes through there are some things missing with which we'll be grateful to get some guidance:

  • docs and tests
  • keeping retrieved access tokens in the db
  • removing tokens when removing shares (will depend on the previous point)

Also this depends on a pending PR for Guzzle:

guzzle/guzzle#3310

@enriquepabloenriquepablo requested a review froma team as acode ownerDecember 17, 2025 16:33
@enriquepabloenriquepablo requested review fromAltahrim,ArtificialOwl,CarlSchwan andsalmart-dev and removed request fora teamDecember 17, 2025 16:33
useSabre\DAV\Xml\Property\ResourceType;
useSabre\HTTP\ClientException;
useSabre\HTTP\ClientHttpException;
useSabre\HTTP\RequestInterface;
Copy link
Author

@enriquepabloenriquepabloDec 17, 2025•
edited
Loading

Choose a reason for hiding this comment

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

Mistake breaking up the code into commits, the missing open-comment is added in a later commit:

bcf7ea1#diff-53ee4c908f9ae3a27e45d03c57ac5ed83c5916d7260561a03f279880671f6155R42

Sorry about this, I'll re-arrange a fixed pr asap.

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@ArtificialOwlArtificialOwlAwaiting requested review from ArtificialOwlArtificialOwl is a code owner automatically assigned from nextcloud/server-backend

@AltahrimAltahrimAwaiting requested review from AltahrimAltahrim is a code owner automatically assigned from nextcloud/server-backend

@salmart-devsalmart-devAwaiting requested review from salmart-devsalmart-dev is a code owner automatically assigned from nextcloud/server-backend

@CarlSchwanCarlSchwanAwaiting requested review from CarlSchwanCarlSchwan is a code owner automatically assigned from nextcloud/server-backend

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@enriquepablo
[8]ページ先頭
©2009-2025 Movatter.jp