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 to v8-beta.7 #1676

Merged
merged 3 commits into from
Sep 29, 2023
Merged

Conversation

Pytal
Copy link
Member

@Pytal Pytal commented Sep 29, 2023

  • Upgrade v7 → v8-beta
  • Adapt to breaking changes
  • Smoke test

nickvergessen and others added 3 commits September 29, 2023 09:02
Signed-off-by: Christopher Ng <[email protected]>
Signed-off-by: Joas Schilling <[email protected]>
Signed-off-by: Christopher Ng <[email protected]>
@nickvergessen nickvergessen force-pushed the deps/nextcloud/vue-8.0.0-beta.7 branch from 3568e17 to fca0f20 Compare September 29, 2023 07:03
@nickvergessen
Copy link
Member

Thanks, I cleaned up the package-locked.json by replacing your commit with:

npm remove @nextcloud/vue
npm i -S @nextcloud/vue@^8.0.0-beta.7

That updated some more depedencies:

-/*! @license DOMPurify 2.4.7 | (c) Cure53 and other contributors | Released under the Apache license 2.0 and Mozilla Public License 2.0 | github.com/cure53/DOMPurify/blob/2.4.7/LICENSE */
+/*! @license DOMPurify 3.0.6 | (c) Cure53 and other contributors | Released under the Apache license 2.0 and Mozilla Public License 2.0 | github.com/cure53/DOMPurify/blob/3.0.6/LICENSE */

…

-* focus-trap 7.5.2
+* focus-trap 7.5.3

Copy link
Collaborator

@Antreesy Antreesy left a comment

Choose a reason for hiding this comment

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

Smoke tested, also checked changes for used components stable7 / master
Looks good, thanks!

@Pytal Pytal merged commit c639fe0 into master Sep 29, 2023
19 checks passed
@Pytal Pytal deleted the deps/nextcloud/vue-8.0.0-beta.7 branch September 29, 2023 23:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants