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

Bump @nextcloud/vue to 6.0.0-beta.3 #2794

Merged
merged 14 commits into from
Aug 18, 2022
Merged

Conversation

juliushaertl
Copy link
Member

Still throwing some errors in the viewer as nextcloud/viewer#1319 / nextcloud/viewer#1300 are missing and the Vue instance has a different version there.

Ref vuejs/vue#12627

At some point we should probably rethink our way of embedding text in viewer to avoid version dependencies like that.

@juliushaertl juliushaertl added the dependencies Pull requests that update a dependency file label Aug 11, 2022
@juliushaertl juliushaertl added this to the Nextcloud 25 milestone Aug 11, 2022
@PVince81 PVince81 closed this Aug 11, 2022
@PVince81 PVince81 reopened this Aug 11, 2022
@vinicius73
Copy link
Member

viewer has been updated to vue 2.7

nextcloud/viewer#1300

There no more errors like vuejs/vue#12627

@vinicius73
Copy link
Member

I just found a style error, it seems to work correctly.

The main problem is the new popover component (floating-vue).
There are some behavior making the tests do not work anymore.

I will send a PR to nextcloud-vue to address some of that behaviors.

@juliushaertl juliushaertl force-pushed the dependencies/nextcloud-vue6 branch 2 times, most recently from a253e31 to c32eae4 Compare August 12, 2022 08:06
@juliushaertl
Copy link
Member Author

@vinicius73 I pushed an adjustment to the cypress selectors and it seems to be happy about that. Also ran another smoke test an couldn't find any obvious issues. Maybe you can later list your findings, so we have that tracked.

Voting for getting this in fast, we can still follow up on smaller issues.

Copy link
Contributor

@Raudius Raudius left a comment

Choose a reason for hiding this comment

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

Tested against server master branch, working well! :)

@juliushaertl
Copy link
Member Author

juliushaertl and others added 11 commits August 18, 2022 23:10
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Bumps [@nextcloud/eslint-config](https://github.com/nextcloud/eslint-config) from 8.0.0 to 8.1.2.
- [Release notes](https://github.com/nextcloud/eslint-config/releases)
- [Changelog](https://github.com/nextcloud/eslint-config/blob/master/CHANGELOG.md)
- [Commits](nextcloud-libraries/eslint-config@v8.0.0...v8.1.2)

---
updated-dependencies:
- dependency-name: "@nextcloud/eslint-config"
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Vinicius Reis <vinicius.reis@nextcloud.com>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
@juliushaertl juliushaertl force-pushed the dependencies/nextcloud-vue6 branch 2 times, most recently from 2a7654d to ff3f55e Compare August 18, 2022 21:51
Signed-off-by: Julius Härtl <jus@bitgrid.net>
@juliushaertl
Copy link
Member Author

/compile

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@juliushaertl juliushaertl merged commit 9ac8198 into master Aug 18, 2022
@delete-merged-branch delete-merged-branch bot deleted the dependencies/nextcloud-vue6 branch August 18, 2022 23:04
@juliushaertl juliushaertl changed the title Bump @nextcloud/vue to 6.0.0-beta.1 Bump @nextcloud/vue to 6.0.0-beta.3 Aug 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants