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

chore: Bump @nextcloud/vue + @nextcloud/dialogs + @nextcloud/calendar-availability-vue #40692

Merged
merged 5 commits into from Oct 6, 2023

Conversation

Pytal
Copy link
Member

@Pytal Pytal commented Sep 29, 2023

Summary

  • Bump @nextcloud/vue + @nextcloud/dialogs + @nextcloud/calendar-availability-vue
  • Adapt to breaking changes
  • Fix acceptance tests

To Do

  • Resolve bundler CI errors

Checklist

@st3iny
Copy link
Member

st3iny commented Oct 5, 2023

ERROR in ./node_modules/@nextcloud/dialogs/dist/chunks/FilePicker-fe2ce33f.mjs 155:217-219
export 'NcDatetime' (imported as 'ee') was not found in '@nextcloud/vue'

It seems that dialogs (and probably any other dependency that makes use of nextcloud/vue) needs to updated as well. It should be NcDateTime.

Ref nextcloud-libraries/nextcloud-vue#4560

@ShGKme
Copy link
Contributor

ShGKme commented Oct 5, 2023

Both @nextcloud/calendar-availability-vue and @nextcloud/dialogs updates were published, so we can upgrade @nextcloud/vue now

@Pytal Pytal force-pushed the deps/nextcloud/vue-8.0.0-beta.7 branch from 4eec177 to e2ced9d Compare October 5, 2023 16:39
@Pytal Pytal marked this pull request as ready for review October 5, 2023 16:41
@Pytal Pytal added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Oct 5, 2023
@Pytal Pytal changed the title chore: Bump @nextcloud/vue to v8-beta.7 chore: Bump @nextcloud/vue + @nextcloud/dialogs + @nextcloud/calendar-availability-vue Oct 5, 2023
Copy link
Contributor

@ShGKme ShGKme left a comment

Choose a reason for hiding this comment

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

🎉

@ShGKme
Copy link
Contributor

ShGKme commented Oct 6, 2023

crying

Working under a fix for tests

…-availability-vue

Signed-off-by: Christopher Ng <chrng8@gmail.com>
Signed-off-by: Christopher Ng <chrng8@gmail.com>
Signed-off-by: Christopher Ng <chrng8@gmail.com>
Signed-off-by: Christopher Ng <chrng8@gmail.com>
@Pytal Pytal force-pushed the deps/nextcloud/vue-8.0.0-beta.7 branch from 0cd8007 to 4f1751f Compare October 6, 2023 21:43
Signed-off-by: Christopher Ng <chrng8@gmail.com>
@Pytal Pytal added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Oct 6, 2023
@Pytal Pytal merged commit 5979651 into master Oct 6, 2023
39 of 40 checks passed
@Pytal Pytal deleted the deps/nextcloud/vue-8.0.0-beta.7 branch October 6, 2023 22:32
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 dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants