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.

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

Remove deprecation on open method #936

Merged
merged 1 commit into from
Aug 2, 2021
Merged

Remove deprecation on open method #936

merged 1 commit into from
Aug 2, 2021

Conversation

skjnldsv
Copy link
Member

@skjnldsv skjnldsv commented Jun 5, 2021

Signed-off-by: John Molakvoæ (skjnldsv) skjnldsv@protonmail.com

@skjnldsv
Copy link
Member Author

skjnldsv commented Jun 5, 2021

/compile amend /

@skjnldsv skjnldsv added 3. to review Waiting for reviews enhancement New feature or request labels Jun 5, 2021
@skjnldsv skjnldsv self-assigned this Jun 5, 2021
@skjnldsv skjnldsv added this to the Nextcloud 22 milestone Jun 5, 2021
@beardhatcode
Copy link
Contributor

Has been there for 17 months (b313993). Looks good to me.

@beardhatcode
Copy link
Contributor

/compile amend /

@azul
Copy link
Contributor

azul commented Jun 5, 2021

Makes sense to me. These are the uses of the old syntax i found:

Also... maybe this should be noted in the app upgrade guide

azul added a commit to nextcloud/officeonline that referenced this pull request Jun 5, 2021
With nextcloud/viewer#936 and Nextcloud 22 the old syntax will not be supported anymore.
azul added a commit to nextcloud/officeonline that referenced this pull request Jun 5, 2021
With nextcloud/viewer#936 and Nextcloud 22 the old syntax will not be supported anymore.

Signed-off-by: Azul <azul@riseup.net>
azul added a commit to nextcloud/recommendations that referenced this pull request Jun 5, 2021
With nextcloud/viewer#936 and Nextcloud 22 the old syntax will not be supported anymore.

Signed-off-by: Azul <azul@riseup.net>
azul added a commit to nextcloud/deck that referenced this pull request Jun 5, 2021
With nextcloud/viewer#936 and Nextcloud 22 the old syntax will not be supported anymore.

Signed-off-by: Azul <azul@riseup.net>
@skjnldsv
Copy link
Member Author

skjnldsv commented Jun 5, 2021

Makes sense to me. These are the uses of the old syntax i found:

@juliushaertl

@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 Jun 5, 2021
@skjnldsv
Copy link
Member Author

skjnldsv commented Jun 5, 2021

@beardhatcode

/compile amend /

This bot does not rebase 😉

@skjnldsv skjnldsv added 3. to review Waiting for reviews and removed 4. to release Ready to be released and/or waiting for tests to finish labels Jun 5, 2021
@skjnldsv skjnldsv requested a review from juliusknorr June 5, 2021 16:23
npmbuildbot-nextcloud bot pushed a commit to nextcloud/recommendations that referenced this pull request Jun 5, 2021
With nextcloud/viewer#936 and Nextcloud 22 the old syntax will not be supported anymore.

Signed-off-by: Azul <azul@riseup.net>
Signed-off-by: npmbuildbot-nextcloud[bot] <npmbuildbot-nextcloud[bot]@users.noreply.github.com>
@juliusknorr
Copy link
Member

Needs another rebase 🙈

@juliusknorr
Copy link
Member

Makes sense to me. These are the uses of the old syntax i found

All replaced already thanks to @azul 🚀

@blizzz
Copy link
Member

blizzz commented Jun 23, 2021

conflicts

@skjnldsv
Copy link
Member Author

skjnldsv commented Aug 2, 2021

/compile amend /

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
Signed-off-by: npmbuildbot-nextcloud[bot] <npmbuildbot-nextcloud[bot]@users.noreply.github.com>
@skjnldsv skjnldsv merged commit 2356dc9 into master Aug 2, 2021
@skjnldsv skjnldsv deleted the fix/open-deprecation branch August 2, 2021 08:37
@skjnldsv skjnldsv added the pending documentation Pending a documentation update label Aug 2, 2021
thaitv21 pushed a commit to VTS-WORKSPACE/recommendations that referenced this pull request Mar 10, 2022
With nextcloud/viewer#936 and Nextcloud 22 the old syntax will not be supported anymore.

Signed-off-by: Azul <azul@riseup.net>
Signed-off-by: npmbuildbot-nextcloud[bot] <npmbuildbot-nextcloud[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews enhancement New feature or request pending documentation Pending a documentation update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants