Skip to content

Conversation

@susnux
Copy link
Contributor

@susnux susnux commented Oct 13, 2025

Summary

This is the first step towards Vue 3.
As this package is a Vue version agnostic version of dialogs, written in Vue 3.

The huge changes count is related to the .license and .map files...

Checklist

@susnux susnux added this to the Nextcloud 33 milestone Oct 13, 2025
@susnux susnux added 3. to review Waiting for reviews dependencies labels Oct 13, 2025
@susnux susnux force-pushed the chore/update-dialog-v7 branch from d31966c to be34bbe Compare October 13, 2025 20:57
This is the first step torwards Vue 3.
As this package is a Vue version agnostic version of dialogs, written in
Vue 3.

Signed-off-by: Ferdinand Thiessen <[email protected]>
@susnux susnux force-pushed the chore/update-dialog-v7 branch 2 times, most recently from 29c9710 to 8ae5a8b Compare October 15, 2025 11:57
Signed-off-by: Ferdinand Thiessen <[email protected]>
@susnux susnux marked this pull request as ready for review October 15, 2025 12:25
@susnux susnux requested review from a team and skjnldsv as code owners October 15, 2025 12:25
@susnux susnux requested review from nfebe and szaimen and removed request for a team October 15, 2025 12:25
@susnux susnux force-pushed the chore/update-dialog-v7 branch from 8ae5a8b to cc4f830 Compare October 15, 2025 12:25
@susnux susnux requested review from CarlSchwan and artonge and removed request for a team October 15, 2025 12:25
@susnux susnux enabled auto-merge October 15, 2025 13:50
})

it('Can move a file to its parent folder', () => {
it.only('Can move a file to its parent folder', () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

Forgot to remove it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

followup: #55791

@susnux susnux merged commit 65cdf14 into master Oct 15, 2025
146 of 151 checks passed
@susnux susnux deleted the chore/update-dialog-v7 branch October 15, 2025 13:51
@max-nextcloud
Copy link
Contributor

It looks like this is breaking text CI. Is it expected that file picker button is not primary anymore?

failed-text-test

@susnux
Copy link
Contributor Author

susnux commented Oct 15, 2025

@max-nextcloud no but that test in text is also not very good, you should not rely on class names.
Better to use roles (@testing-library/cypress with cy.findByRole('button', { name: 'Choose a.png' }))

@susnux
Copy link
Contributor Author

susnux commented Oct 15, 2025

Fix for the primary regression: #55799

max-nextcloud added a commit that referenced this pull request Oct 18, 2025
In Nextcloud Vue v8 the `type` prop for dialog buttons
was deprecated and replaces with the `variant` prop.

In v9 the deprecated props are removed.
Use the `variant` prop in the deprecated `OC.dialogs`.

This is a follow up for #55726.

Signed-off-by: Max <[email protected]>
@susnux susnux mentioned this pull request Oct 22, 2025
6 tasks
@susnux
Copy link
Contributor Author

susnux commented Dec 16, 2025

/backport to stable32

susnux pushed a commit that referenced this pull request Dec 16, 2025
In Nextcloud Vue v8 the `type` prop for dialog buttons
was deprecated and replaces with the `variant` prop.

In v9 the deprecated props are removed.
Use the `variant` prop in the deprecated `OC.dialogs`.

This is a follow up for #55726.

Signed-off-by: Max <[email protected]>
susnux pushed a commit that referenced this pull request Dec 17, 2025
In Nextcloud Vue v8 the `type` prop for dialog buttons
was deprecated and replaces with the `variant` prop.

In v9 the deprecated props are removed.
Use the `variant` prop in the deprecated `OC.dialogs`.

This is a follow up for #55726.

Signed-off-by: Max <[email protected]>
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 dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants