Skip to content

Conversation

@dependabot
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] <[email protected]>
Signed-off-by: nextcloud-command <[email protected]>
@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
<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.

3 participants