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: shared views can be favorited #921

Merged
merged 2 commits into from
Mar 18, 2024
Merged

fix: shared views can be favorited #921

merged 2 commits into from
Mar 18, 2024

Conversation

elzody
Copy link
Contributor

@elzody elzody commented Mar 11, 2024

Current behavior

If a view is shared (the user is not the owner of the view), the user will have the option to favorite/unfavorite it, but this will not be reflected in the UI (it will not be added/removed from the favorites list).

Suggested behavior

Fix the behavior defined above. Additionally, if a table that contains views is favorited, its views will then not display a favorite/unfavorite button until the table is unfavorited. Any views which are already favorited but are also a part of the table will disappear from the favorite list as an individual view, and instead appear normally under the favorited table. This is done to prevent the view from showing in the favorites list twice.

Recording

2024-03-11.12-55-48.mp4

TODO

  • Determine if this behavior is appropriate
  • Address any design concerns
  • Implement testing

Copy link
Member

@nimishavijay nimishavijay left a comment

Choose a reason for hiding this comment

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

Looks good! I would only suggest that the unfavoriting action for a view still be shown even if it's under a favorited table, in case people want to unfavorite individual views on a favorited table. Then it would also follow a consistent pattern of "everything that's not favorited can be favorited and everything that is favorited can be unfavorited". Does that make sense? :)

@elzody
Copy link
Contributor Author

elzody commented Mar 11, 2024

Looks good! I would only suggest that the unfavoriting action for a view still be shown even if it's under a favorited table, in case people want to unfavorite individual views on a favorited table. Then it would also follow a consistent pattern of "everything that's not favorited can be favorited and everything that is favorited can be unfavorited". Does that make sense? :)

Yes, absolutely. Thank you!

@juliusknorr juliusknorr added bug Something isn't working 3. to review Waiting for reviews labels Mar 11, 2024
@juliusknorr juliusknorr added this to the v0.7.0 milestone Mar 11, 2024
Signed-off-by: Elizabeth Danzberger <lizzy7128@tutanota.de>
Copy link
Contributor

@enjeck enjeck left a comment

Choose a reason for hiding this comment

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

Cool 😎 💯

cypress/e2e/tables-favorite.cy.js Outdated Show resolved Hide resolved
Signed-off-by: Elizabeth Danzberger <lizzy7128@tutanota.de>
@elzody elzody enabled auto-merge (squash) March 18, 2024 20:34
@elzody elzody merged commit 7f4fa63 into main Mar 18, 2024
45 checks passed
@elzody elzody deleted the fix/view-fav-perms branch March 18, 2024 20:42
Copy link
Contributor

Hello there,
Thank you so much for taking the time and effort to create a pull request to our Nextcloud project.

We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process.

Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6

Thank you for contributing to Nextcloud and we hope to hear from you soon!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews bug Something isn't working feedback-requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants