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

build(deps): bump @nextcloud/vue from 4.3.0 to 5.0.0 #1159

Merged
merged 1 commit into from
Feb 24, 2022

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Feb 12, 2022

Bumps @nextcloud/vue from 4.3.0 to 5.0.0.

Release notes

Sourced from @​nextcloud/vue's releases.

v5.0.0

v5.0.0 (2022-02-10)

Full Changelog

Breaking changes

  • CounterBubble: Removed highlighted prop #2208

Enhancements

Fixed bugs

Closed pull requests

Changelog

Sourced from @​nextcloud/vue's changelog.

v5.0.0 (2022-02-10)

Full Changelog

Enhancements

Fixed bugs

Closed pull requests

Commits
  • 6c7ef0b Merge pull request #2488 from nextcloud/v5.0.0
  • d7bb77c Bump version to v5.0.0
  • 9c2688f Merge pull request #2486 from nextcloud/bugfix/2484/limit-avatar-requests
  • 6b85dc9 Merge pull request #2482 from nextcloud/dependabot/npm_and_yarn/jest-27.5.1
  • 5e4e8e4 Bump jest from 27.5.0 to 27.5.1
  • 29d3ac9 Merge pull request #2470 from nextcloud/translations_l10n-messages-pot--maste...
  • 32dea84 Merge pull request #2487 from nextcloud/translations_l10n-messages-pot--maste...
  • 32a996d Translate /l10n/messages.pot in pl
  • 1e54980 Limit avatar request sizes to 64 and 512 pixel
  • cd0a9e3 Merge pull request #2483 from nextcloud/fix/5987-create_submailbox_in_submail...
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

@dependabot dependabot bot added 3. to review Waiting for reviews dependencies Pull requests that update a dependency file labels Feb 12, 2022
@skjnldsv
Copy link
Member

/compile amend /

@nextcloud-command nextcloud-command force-pushed the dependabot/npm_and_yarn/nextcloud/vue-5.0.0 branch from ec31f7b to 87e825d Compare February 22, 2022 07:24
@skjnldsv
Copy link
Member

@dependabot recreate

@dependabot dependabot bot force-pushed the dependabot/npm_and_yarn/nextcloud/vue-5.0.0 branch from 87e825d to b34ab5f Compare February 22, 2022 07:28
@skjnldsv
Copy link
Member

/rebase

@nextcloud-command nextcloud-command force-pushed the dependabot/npm_and_yarn/nextcloud/vue-5.0.0 branch from b34ab5f to cb30adc Compare February 22, 2022 08:45
@skjnldsv
Copy link
Member

/rebase

@nextcloud-command nextcloud-command force-pushed the dependabot/npm_and_yarn/nextcloud/vue-5.0.0 branch from cb30adc to adc36bd Compare February 22, 2022 09:03
@skjnldsv
Copy link
Member

/compile amend /

@nextcloud-command nextcloud-command force-pushed the dependabot/npm_and_yarn/nextcloud/vue-5.0.0 branch from adc36bd to 472e705 Compare February 22, 2022 09:12
Copy link
Contributor

@szaimen szaimen left a comment

Choose a reason for hiding this comment

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

I guess we should verify if the modal sizes still look good?

@szaimen szaimen self-assigned this Feb 23, 2022
@szaimen szaimen force-pushed the dependabot/npm_and_yarn/nextcloud/vue-5.0.0 branch from a0626d3 to 497f12d Compare February 23, 2022 15:58
@szaimen
Copy link
Contributor

szaimen commented Feb 23, 2022

/compile amend /

Copy link
Contributor

@szaimen szaimen left a comment

Choose a reason for hiding this comment

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

lgtm now 👍
Verified with a mp4, flac and jpg file

@szaimen szaimen added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Feb 23, 2022
Bumps [@nextcloud/vue](https://github.com/nextcloud/nextcloud-vue) from 4.3.0 to 5.0.0.
- [Release notes](https://github.com/nextcloud/nextcloud-vue/releases)
- [Changelog](https://github.com/nextcloud/nextcloud-vue/blob/master/CHANGELOG.md)
- [Commits](nextcloud-libraries/nextcloud-vue@v4.3.0...v5.0.0)

---
updated-dependencies:
- dependency-name: "@nextcloud/vue"
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@nextcloud-command nextcloud-command force-pushed the dependabot/npm_and_yarn/nextcloud/vue-5.0.0 branch from 497f12d to 281625d Compare February 23, 2022 16:07
@szaimen
Copy link
Contributor

szaimen commented Feb 23, 2022

@skjnldsv do you mind fixing cypress? (Not sure how to do that)

@szaimen szaimen assigned skjnldsv and unassigned szaimen Feb 23, 2022
@skjnldsv
Copy link
Member

/rebase

@skjnldsv
Copy link
Member

cypress-video5f97c7af-51b6-46bf-a341-dba1a2e974d1.mp4

@skjnldsv
Copy link
Member

skjnldsv commented Feb 23, 2022

To check if the selectors are the same wince the last update of @nc/vue

  Download image1.jpg from viewer in link share
    ✓ See the default files list (36172ms)
    ✓ Does not have any visual regression 1 (43ms)
    ✓ See shared files in the list (766ms)
    ✓ Does not have any visual regression 2 (68ms)
    ✓ Share the Photos folder with a share link and access the share link (6968ms)
    ✓ Does not have any visual regression 3 (283ms)
    ✓ Open the viewer on file click (708ms)
    ✓ Does not see a loading animation (875ms)
    (Attempt 1 of 3) See the download icon and title on the viewer header
    (Attempt 2 of 3) See the download icon and title on the viewer header
    1) See the download icon and title on the viewer header
    (Attempt 1 of 3) Download the image
    (Attempt 2 of 3) Download the image
    2) Download the image
    (Attempt 1 of 3) Compare downloaded file with asset by size
    (Attempt 2 of 3) Compare downloaded file with asset by size
    3) Compare downloaded file with asset by size


  8 passing (2m)
  3 failing

  1) Download image1.jpg from viewer in link share
       See the download icon and title on the viewer header:
     AssertionError: Timed out retrying after 6000ms: Expected to find element: `body > .viewer .modal-header a.action-item > .download-icon`, but never found it.
      at Context.eval (http://127.0.0.1:8082/__cypress/tests?p=cypress/integration/download-share.spec.js:87:75)

  2) Download image1.jpg from viewer in link share
       Download the image:
     AssertionError: Timed out retrying after 6000ms: Expected to find element: `body > .viewer .modal-header a.action-item > .download-icon`, but never found it.
      at Context.eval (http://127.0.0.1:8082/__cypress/tests?p=cypress/integration/download-share.spec.js:92:8)

  3) Download image1.jpg from viewer in link share
       Compare downloaded file with asset by size:
     AssertionError: Timed out retrying after 5000ms: `cy.readFile("/home/runner/work/viewer/viewer/cypress/downloads/image1.jpg")` failed because the file does not exist at the following path:

`/home/runner/work/viewer/viewer/cypress/downloads/image1.jpg`

https://on.cypress.io/readfile
      at Context.eval (http://127.0.0.1:8082/__cypress/tests?p=cypress/integration/download-share.spec.js:104:10)

@szaimen
Copy link
Contributor

szaimen commented Feb 23, 2022

okay, I guess this is then a bug in vue 5.0? But should not be related to the modal change, as far as I understand...

@szaimen
Copy link
Contributor

szaimen commented Feb 23, 2022

@skjnldsv why is cypress suddenly green?

@skjnldsv
Copy link
Member

skjnldsv commented Feb 24, 2022

@skjnldsv why is cypress suddenly green?

Restarted
Was a false negative, see the video recording freezing and not rendering some icons

@skjnldsv skjnldsv merged commit 8cf5187 into master Feb 24, 2022
@skjnldsv skjnldsv deleted the dependabot/npm_and_yarn/nextcloud/vue-5.0.0 branch February 24, 2022 07:03
@@ -24,6 +24,7 @@
<template>
<Modal v-if="initiated || currentFile.modal"
id="viewer"
size="large"
Copy link
Member

Choose a reason for hiding this comment

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

This should have been full, this is a regression

Before After
2022-04-29_08-43 2022-04-29_08-43_1

Copy link
Contributor

Choose a reason for hiding this comment

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

Oh I see. Do you want to create a pr with a fix or shall I?

Copy link
Member

Choose a reason for hiding this comment

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

I'm on it, there are lots of issues we missed :)

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 Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants