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

change wording of the client side storage handling #19815

Open
wants to merge 9 commits into
base: dev
Choose a base branch
from

Conversation

bgruening
Copy link
Member

We are trying to roll out advanced storage configurations on EU. Initial user feedback showed that:

remote file source is confusing. In particular, the source was not clear in cases where you can export to a source. It was also confusing that most of the remote file source were labelled storage.

storage location was not distinct enough to remote file source, so people told us they would expect Dropbox under this section. We considered a few options, but in the end, we think the distinguishing concept is the fact that 1) Galaxy is managing storage (quota, create, delete, permissions) vs. 2) users import and export to an external "thing".

This external "thing" we would like to call a repository from now on in the frontend. We think it's totally fine to call it differently in the backend and be more technical in the backend.

storage location we would like to call Galaxy storage or simply storage (in a few places where the Galaxy prefix does not fit). Galaxy should indicate that the storage is managed by Galaxy. In contrast, we have My repositories.

An additional change is that I removed the word template in the file-source-templates. I think "template" is also a technical artefact that is not relevant to the user.

Please let us know what you think.

@jmchilton
Copy link
Member

I'd love if a UX person looked over this but I think the switch is broadly fine by me. The failing client unit test is also a relevant failure.

@davelopez
Copy link
Contributor

If you are fine with the wording I'll go and try to fix the tests 👍

@davelopez
Copy link
Contributor

The rest of the failures are unrelated and failing in other PRs too.

@bgruening
Copy link
Member Author

THanks a lot @davelopez!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants