Skip to content

Conversation

@skjnldsv
Copy link
Member

@skjnldsv skjnldsv commented Jun 5, 2021

Signed-off-by: John Molakvoæ (skjnldsv) [email protected]

@skjnldsv skjnldsv force-pushed the fix/open-deprecation branch from a0c8087 to 5fb8946 Compare June 5, 2021 09:27
@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 requested review from artonge, azul and beardhatcode June 5, 2021 09:28
@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 <[email protected]>
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 <[email protected]>
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 <[email protected]>
@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 <[email protected]>
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 skjnldsv modified the milestones: Nextcloud 22, Nextcloud 23 Jun 23, 2021
@skjnldsv skjnldsv force-pushed the fix/open-deprecation branch from b3eba95 to 3534262 Compare August 2, 2021 08:23
@skjnldsv
Copy link
Member Author

skjnldsv commented Aug 2, 2021

/compile amend /

Signed-off-by: John Molakvoæ (skjnldsv) <[email protected]>
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 <[email protected]>
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.

6 participants