Skip to content

Conversation

@tcitworld
Copy link
Member

Which was removed in the Vue rewrite in #33728. This breaks things like nextcloud/external#79

Checklist

@tcitworld tcitworld added this to the Nextcloud 26 milestone Dec 8, 2022
@tcitworld tcitworld requested review from a team, Pytal, artonge, juliusknorr and skjnldsv and removed request for a team December 8, 2022 09:06
@tcitworld
Copy link
Member Author

/compile amend /

@nextcloud-command nextcloud-command force-pushed the allow-target-blank-in-app-navigation branch from 802fd57 to e04d042 Compare December 8, 2022 09:53
@skjnldsv skjnldsv added 4. to release Ready to be released and/or waiting for tests to finish regression and removed 3. to review Waiting for reviews labels Dec 8, 2022
@skjnldsv
Copy link
Member

skjnldsv commented Dec 8, 2022

conflicts :(

@tcitworld
Copy link
Member Author

/compile amend /

@nextcloud-command nextcloud-command force-pushed the allow-target-blank-in-app-navigation branch from e04d042 to a54d82b Compare December 8, 2022 15:40
@PVince81
Copy link
Member

@tcitworld needs manual rebase

@nextcloud-command nextcloud-command force-pushed the allow-target-blank-in-app-navigation branch from a54d82b to e97ef64 Compare December 16, 2022 17:11
@tcitworld tcitworld force-pushed the allow-target-blank-in-app-navigation branch from e97ef64 to bb6de2a Compare January 4, 2023 17:04
@skjnldsv skjnldsv changed the title Allow target="_blank" in app navigation items Allow target="_blank" in apps navigation items Jan 4, 2023
@nickvergessen
Copy link
Member

Conflicting files
dist/core-main.js
dist/core-main.js.map

:(

There are quite some PRs in the queue atm.

@tcitworld tcitworld force-pushed the allow-target-blank-in-app-navigation branch from bb6de2a to ef2ccfc Compare January 4, 2023 17:22
Which was removed in the Vue rewrite in #33728. This breaks things like nextcloud/external#79

Signed-off-by: Thomas Citharel <[email protected]>
Signed-off-by: Joas Schilling <[email protected]>
@nickvergessen nickvergessen force-pushed the allow-target-blank-in-app-navigation branch from ef2ccfc to 5bcc277 Compare January 5, 2023 06:14
@nickvergessen nickvergessen merged commit 77c016b into master Jan 5, 2023
@nickvergessen nickvergessen deleted the allow-target-blank-in-app-navigation branch January 5, 2023 06:39
@nextcloud nextcloud deleted a comment from tcitworld Jan 5, 2023
@nickvergessen
Copy link
Member

Manual backport in #36007

@backportbot-nextcloud
Copy link

The backport to stable25 failed. Please do this backport manually.

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

Labels

4. to release Ready to be released and/or waiting for tests to finish feature: vue apps javascript regression

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants