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] feat: update image view width and add title #4165

Merged
merged 2 commits into from Jun 8, 2023

Conversation

backportbot-nextcloud[bot]
Copy link

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

  • ⚠️ This backport had conflicts that were resolved with the 'ours' merge strategy and is likely incomplete

backport of #4070

@backportbot-nextcloud backportbot-nextcloud bot added the bug Something isn't working label May 10, 2023
@backportbot-nextcloud backportbot-nextcloud bot added this to the Nextcloud 26.0.2 milestone May 10, 2023
@juliushaertl
Copy link
Member

@luka-nextcloud Please double check if the backport is missing any code change outside of the generated js bundles.

@cypress
Copy link

cypress bot commented May 10, 2023

2 flaky tests on run #10085 ↗︎

0 143 1 0 Flakiness 2

Details:

[stable26] feat: update image view width and add title
Project: Text Commit: 31d3575e8f
Status: Passed Duration: 03:29 💡
Started: Jun 8, 2023 7:20 AM Ended: Jun 8, 2023 7:24 AM
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
Flakiness  conflict.spec.js • 1 flaky test

View Output Video

Test Artifacts
Open test.md in viewer > resolves conflict using server version Output Screenshots

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

@luka-nextcloud
Copy link
Contributor

@juliushaertl It's not missing any code. Let me see why the cypress test is failing.

@blizzz
Copy link
Member

blizzz commented May 17, 2023

appears to be in dev, moving to 26.0.3

@juliushaertl juliushaertl marked this pull request as ready for review May 17, 2023 15:09
Signed-off-by: Luka Trovic <luka@nextcloud.com>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@mejo-
Copy link
Member

mejo- commented Jun 8, 2023

/compile

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@mejo- mejo- merged commit 082b575 into stable26 Jun 8, 2023
32 checks passed
@delete-merged-branch delete-merged-branch bot deleted the backport/4070/stable26 branch June 8, 2023 07:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants