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

Move remnants of ocs api requests to v2 endpoint #26466

Merged
merged 1 commit into from
May 26, 2021
Merged

Move remnants of ocs api requests to v2 endpoint #26466

merged 1 commit into from
May 26, 2021

Conversation

skjnldsv
Copy link
Member

@skjnldsv skjnldsv commented Apr 9, 2021

Upcoming @nextcloud/router update move the default endpoint to v2, until then, we have to specify it

@skjnldsv skjnldsv added bug 3. to review Waiting for reviews labels Apr 9, 2021
@skjnldsv skjnldsv added this to the Nextcloud 21.0.1 milestone Apr 9, 2021
@skjnldsv skjnldsv self-assigned this Apr 9, 2021
@skjnldsv
Copy link
Member Author

skjnldsv commented Apr 9, 2021

/backport to stable21

@skjnldsv
Copy link
Member Author

skjnldsv commented Apr 9, 2021

/compile amend /

@skjnldsv skjnldsv requested a review from nickvergessen April 9, 2021 12:24
@MorrisJobke MorrisJobke mentioned this pull request May 20, 2021
Merged
@MorrisJobke
Copy link
Member

Please resolve conflicts

MorrisJobke
MorrisJobke approved these changes May 21, 2021
View reviewed changes
Copy link
Member

@MorrisJobke MorrisJobke left a comment

Choose a reason for hiding this comment

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

Code looks good 👍

@skjnldsv
Move remnants of ocs api requests to v2 endpoint
f1f8fa1 
Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
@skjnldsv skjnldsv added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels May 26, 2021
@skjnldsv skjnldsv merged commit 903b99b into master May 26, 2021
@skjnldsv skjnldsv deleted the fix/ocs-api-v2 branch May 26, 2021 06:59
@backportbot-nextcloud backportbot-nextcloud bot mentioned this pull request May 26, 2021
Merged
@MorrisJobke MorrisJobke mentioned this pull request May 26, 2021
Merged
98 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MorrisJobke MorrisJobke MorrisJobke approved these changes

@ChristophWurst ChristophWurst ChristophWurst approved these changes

@rullzer rullzer Awaiting requested review from rullzer

@PVince81 PVince81 Awaiting requested review from PVince81

@juliusknorr juliusknorr Awaiting requested review from juliusknorr

@nickvergessen nickvergessen Awaiting requested review from nickvergessen

Assignees

@skjnldsv skjnldsv

Labels
4. to release Ready to be released and/or waiting for tests to finish bug
Projects
None yet
Milestone
Nextcloud 22
Development

Successfully merging this pull request may close these issues.
4 participants
@skjnldsv @MorrisJobke @nickvergessen @ChristophWurst