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
UI for creating & managing shares for tables and explorations #3127
Conversation
…ci/mathesar into share_creation_ui
@ghislaineguerin and @kgodey I've requested your reviews for the UX for sharing tables & explorations. Please take a look at the video for demonstration of the feature. |
This PR is marked as a draft because it depends on another PR which isn't merged it. But it is ready for review. |
… share_creation_ui
…ci/mathesar into share_creation_ui
…ci/mathesar into share_creation_ui
…ci/mathesar into share_creation_ui
Thank you, @pavish . I think this is pretty straightforward. I don't see anything at this point that needs to be changed. I believe the options to create a link, and then to disable and regenerate, are very easy to understand. Let's please ensure that we're using title case in the action buttons, as I believe we do that elsewhere. If you look at the 'Links' section in the inspector, we have 'Create Link,' where 'Link' is in uppercase. I also don't know if we want (and this is something just to consider) to show somewhere in the table properties that this table is currently shared. The only way to know the status is to open the share button. |
@pavish The UI looks good to me, nice work! Like @ghislaineguerin, I also have a question about what the "Share" button will look like when there's already a link created. I assume it will change state somehow to show that the table / exploration is shared. I'd like to see that too. |
Re-assigned to @rajatvijay for review per discussion. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Backend changes approved.
Thanks for your reviews.
This is currently not handled. The user will only know if the table is already shared or not when they open the dropdown. I thought of a change in color, or a dot notification, both of which require a UX pattern which we haven't established uniformly, and I ultimately decided to get the feature completed and handle this later. We can handle this as a separate issue for the next release. Does that sound good?
Yes, thanks Ghislaine. I'll make these changes. |
Yes, but please make an issue to track it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚀
Fixes #3035
This PR
Demo
Demo of UX for Share Table and Share Exploration Functionality - Watch Video
Screenshots
Flow: Sharing an entity
When table isn't shared
After sharing the table
Flow: Viewing shared entity using the link
Table
Exploration
Checklist
Update index.md
).develop
branch of the repositoryvisible errors.
Developer Certificate of Origin
Developer Certificate of Origin