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

[full-ci] Improve error message for internal links on unaccepted shares #7814

Merged
merged 4 commits into from
Oct 18, 2022

Conversation

JammingBen
Copy link
Contributor

@JammingBen JammingBen commented Oct 17, 2022

Description

The error message when resolving an internal link to an unaccepted share has been improved for a better UX.

Also, the resolving page has been redesigned.

Screenshots

image

Related Issue

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests

@JammingBen JammingBen self-assigned this Oct 17, 2022
@ownclouders
Copy link
Contributor

ownclouders commented Oct 17, 2022

Results for oCISBasic https://drone.owncloud.com/owncloud/web/29148/52/1

💥 The acceptance tests failed on retry. Please find the screenshots inside ...

webUIPrivateLinks-accessingPrivateLinks_feature-L35.png

webUIPrivateLinks-accessingPrivateLinks_feature-L35.png

@JammingBen JammingBen force-pushed the private-link-on-unaccepted-share branch from 1d50f4d to a1d7d8f Compare October 18, 2022 08:23
@JammingBen JammingBen changed the title Improve error message for internal links on unaccepted shares [full-ci+] Improve error message for internal links on unaccepted shares Oct 18, 2022
@JammingBen JammingBen changed the title [full-ci+] Improve error message for internal links on unaccepted shares [full-ci] Improve error message for internal links on unaccepted shares Oct 18, 2022
@JammingBen JammingBen force-pushed the private-link-on-unaccepted-share branch from a1d7d8f to 025542d Compare October 18, 2022 08:45
@dschmidt
Copy link
Member

Just a nitpick, but I'm not really a fan of the filename as a centered headline but still being a part of the following text.
I think I would prefer a full sentence like `This file has been shared with you" or something like that

@tbsbdr thoughts?

@tbsbdr
Copy link

tbsbdr commented Oct 18, 2022

@janackermann might have had the same thought though you wondered if it works with different languages.

Lets change it to:

Love Letter.md

This item has been shared with you ...

note: If you have the possibility, a more specific "This file... " or "This folder ..." would be better than just "item"

thx @dschmidt for raising feedback

@sonarqubecloud
Copy link

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

3.4% 3.4% Coverage
0.0% 0.0% Duplication

@JammingBen JammingBen merged commit 941f2bf into master Oct 18, 2022
@delete-merged-branch delete-merged-branch bot deleted the private-link-on-unaccepted-share branch October 18, 2022 11:58
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.

Internal link: Receiver sees error if link is not accepted
4 participants