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

Hide buttons that either won't work or won't give user notification on failure #416

Merged
merged 2 commits into from Feb 20, 2019

Conversation

bodom0015
Copy link
Member

@bodom0015 bodom0015 commented Feb 20, 2019

Problem

These functions currently have outstanding problems that will not be fixed for this release.

Fixes #409
Fixes #415

Approach

We should hide them from the Tale Workspaces UI for now, until they can be investigated and fixed.

How to Test

  1. Checkout and run this branch locally
  2. Login to the WholeTale Dashboard
  3. Navigate to "Browse" and hover over a Tale to which you do not have access
  4. Switch to Cards view and hover over the image (first column) of a Tale to which you do not have access
  5. Launch a Tale to which you have access (if you haven't already)
  6. Navigate to Run > Files > Tale Workspaces
  7. Create a folder or file (if you haven't already)
  8. Expand the dropdown next to the folder or file

@bodom0015 bodom0015 changed the title Hide buttons that either won't work or give user notification on failure Hide buttons that either won't work or won't give user notification on failure Feb 20, 2019
@Xarthisius Xarthisius merged commit 2e48790 into master Feb 20, 2019
@Xarthisius Xarthisius deleted the hide-copy-to-home-from-workspaces branch February 27, 2019 19:47
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.

Hide "Move To..." / "Copy to Home" in Tale Workspaces view Delete enabled on public Tale
2 participants