feature/move-files-styling

[BrianWhitneyAI]

The purpose of this PR is to resolve #315 and clean up the "Copy files" modal. This PR only resolves some of the items as UX changes related to to some of the requests are dependent on future ETL work.

[BrianWhitneyAI]

[BrianWhitneyAI]

Could definitely use some help with the wording here. Spacing is also not great.

[pgarrison]

I suggest not using "Cache" or "lease": I think these terms will be unfamiliar in this context. Perhaps the following instead:

Copy Files to Local NAS (Vast)

Files copied to the local NAS (Vast) are stored with a 180-day expiration, after which they revert to cloud-only. To extend the expiration, simply reselect the files and confirm the update.

Files that are already on Vast: Extend expiration

Files to download to Vast

[BrianWhitneyAI]

I suggest not using "Cache" or "lease": I think these terms will be unfamiliar in this context. Perhaps the following instead:

Copy Files to Local NAS (Vast)

Files copied to the local NAS (Vast) are stored with a 180-day expiration, after which they revert to cloud-only. To extend the expiration, simply reselect the files and confirm the update.

Files that are already on Vast: Extend expiration

Files to download to Vast

resolved in a94c95e

[SeanLeRoy]

LGTM assuming we add the total file size count to the bottom of both tables as discussed in slack

[lynwilhelm]

Thanks for requesting my eyes @pgarrison. Your suggestions were right on and Brian and I chatted on a couple additional minor edits that should now be resolved.

[BrianWhitneyAI]

Here are some screenshots!

[saeliddp]

Nit: the expiration/eviction period is currently 182 days, not 180. We can change this on the backend if desired. Could also say '6 months' on the frontend, or maybe it's not an important distinction to the users 🤷 . Just wanted to let you know if you were unaware

Also, glad to see all of my integration test files are being used for something else 😆

[kmitcham]

Glad to see us using the word 'vast' instead of cache in much of the UI, as I think that is more clear to our users.

[BrianWhitneyAI]

Nit: the expiration/eviction period is currently 182 days, not 180. We can change this on the backend if desired. Could also say '6 months' on the frontend, or maybe it's not an important distinction to the users 🤷 . Just wanted to let you know if you were unaware

Also, glad to see all of my integration test files are being used for something else 😆

Those tests have been so helpful! The original text said ~180 days, knowing it was 182. I think its ok to tell users 180 here, Im also ok with 6 months

[BrianWhitneyAI]

Glad to see us using the word 'vast' instead of cache in much of the UI, as I think that is more clear to our users.

Yeah I'm still unsure about "NAS" and if users will know what that is but Vast is now alot clearer.

[pgarrison]

Yeah I'm still unsure about "NAS" and if users will know what that is but Vast is now alot clearer.

I think leaving out "local NAS" entirely makes sense. I don't think our users really know what it means either.