Skip to content

Conversation

@lukasdotcom
Copy link
Member

I was looking at the repo for something else and noticed there wouldn't be much work needed to update to vue 3 and vite

@lukasdotcom lukasdotcom requested a review from julien-nc July 2, 2025 14:44
Copy link
Contributor

@kyteinsky kyteinsky left a comment

Choose a reason for hiding this comment

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

sweet, thanks!

The text seems off here. Would you mind fixing it here and reporting it upstream in the vue library if not already reported (https://github.com/nextcloud-libraries/nextcloud-vue)?

image

@lukasdotcom
Copy link
Member Author

lukasdotcom commented Jul 2, 2025

sweet, thanks!

The text seems off here. Would you mind fixing it here and reporting it upstream in the vue library if not already reported (https://github.com/nextcloud-libraries/nextcloud-vue)?

image

Not sure how to fix it here, but I did add an issue upstream.

@lukasdotcom
Copy link
Member Author

A fix is being made in the library. When 9.0.0-rc4 is out we can merge this.

@lukasdotcom
Copy link
Member Author

@kyteinsky 9.0.0-rc4 for the nextcloud vue libraries came out yesterday. We can now merge this. https://github.com/nextcloud-libraries/nextcloud-vue/releases/tag/v9.0.0-rc.4

Signed-off-by: Lukas Schaefer <lukas@lschaefer.xyz>
Copy link
Contributor

@kyteinsky kyteinsky left a comment

Choose a reason for hiding this comment

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

perfect, thanks!

@lukasdotcom lukasdotcom merged commit 036d2d8 into main Jul 17, 2025
16 checks passed
@lukasdotcom lukasdotcom deleted the vue3 branch July 17, 2025 12:55
@kyteinsky kyteinsky mentioned this pull request Jul 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants