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

fix: Provide file id for single file share links #3846

Merged
merged 2 commits into from
Feb 28, 2023

Conversation

juliushaertl
Copy link
Member

Signed-off-by: Julius Hรคrtl jus@bitgrid.net

๐Ÿ“ Summary

  • Resolves: #

๐Ÿ–ผ๏ธ Screenshots

๐Ÿš๏ธ Before ๐Ÿก After
B A

๐Ÿšง TODO

  • ...

๐Ÿ Checklist

  • Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
  • Sign-off message is added to all commits
  • Tests (unit, integration and/or end-to-end) passing and the changes are covered with tests
  • Documentation (README or documentation) has been updated or is not required

@cypress
Copy link

cypress bot commented Feb 28, 2023

3 flaky tests on run #8790 โ†—๏ธŽ

0 138 0 0 Flakiness 3

Details:

fix: Provide file id for single file share links
Project: Text Commit: c13824cd2b
Status: Passed Duration: 03:46 ๐Ÿ’ก
Started: Feb 28, 2023 9:11 PM Ended: Feb 28, 2023 9:15 PM
Flakinessย  nodes/Links.spec.js โ€ข 1 flaky test

View Output Video

Test
test link marks > link preview > shows a link preview Screenshot
Flakinessย  share.spec.js โ€ข 1 flaky test

View Output Video

Test
Open test.md in viewer > Share a file with download disabled shows an error Screenshot
Flakinessย  sync.spec.js โ€ข 1 flaky test

View Output Video

Test
Save > saves the actual file and document state Screenshot

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

@max-nextcloud max-nextcloud self-assigned this Feb 28, 2023
@max-nextcloud max-nextcloud force-pushed the fix/public-share-fileid branch 2 times, most recently from 8678fda to 75f712f Compare February 28, 2023 21:05
@max-nextcloud max-nextcloud marked this pull request as ready for review February 28, 2023 21:06
juliushaertl and others added 2 commits February 28, 2023 22:07
Signed-off-by: Julius Hรคrtl <jus@bitgrid.net>
Signed-off-by: Max <max@nextcloud.com>
Copy link
Member Author

@juliushaertl juliushaertl left a comment

Choose a reason for hiding this comment

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

๐Ÿ‘

@max-nextcloud max-nextcloud merged commit 0db90d4 into main Feb 28, 2023
@delete-merged-branch delete-merged-branch bot deleted the fix/public-share-fileid branch February 28, 2023 21:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants