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

[stable26] fix(Viewer): remove outdated fix from stable16. #4120

Merged
merged 2 commits into from
May 4, 2023

Conversation

backportbot-nextcloud[bot]
Copy link

@backportbot-nextcloud backportbot-nextcloud bot commented May 4, 2023

  • ⚠️ This backport's changes differ from the original and might be incomplete

backport of #4117

@backportbot-nextcloud backportbot-nextcloud bot added this to the Nextcloud 26.0.1 milestone May 4, 2023
This broke the viewer resize and is not needed anymore.

In order to confirm this is not needed
i replaced the `beforeMount` handler with a `console.info` call
opened a text in the viewer and resized the window a few times.
The provided info only showed up once.
So the `ViewerComponent` was only mounted once as well.

Fixes #4099.

Signed-off-by: Max <[email protected]>
@max-nextcloud max-nextcloud force-pushed the backport/4117/stable26 branch from 21377f5 to eeb2f81 Compare May 4, 2023 08:09
@max-nextcloud
Copy link
Collaborator

/compile

@max-nextcloud
Copy link
Collaborator

confirmed locally that this also doesn't break anything in 26.

Signed-off-by: nextcloud-command <[email protected]>
@cypress
Copy link

cypress bot commented May 4, 2023

2 flaky tests on run #9656 ↗︎

0 142 1 0 Flakiness 2

Details:

[stable26] fix(Viewer): remove outdated fix from stable16.
Project: Text Commit: a5771cd2be
Status: Passed Duration: 04:35 💡
Started: May 4, 2023 11:54 AM Ended: May 4, 2023 11:58 AM
Flakiness  sync.spec.js • 1 flaky test

View Output Video

Test Artifacts
Sync > recovers from a lost connection Output Screenshots
Flakiness  share.spec.js • 1 flaky test

View Output Video

Test Artifacts
Open test.md in viewer > Share a file with download disabled shows an error Output Screenshots

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@juliusknorr juliusknorr merged commit 89f3794 into stable26 May 4, 2023
@delete-merged-branch delete-merged-branch bot deleted the backport/4117/stable26 branch May 4, 2023 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants