-
Notifications
You must be signed in to change notification settings - Fork 1
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
feature/copy-to-local-cache-request #322
feature/copy-to-local-cache-request #322
Conversation
This may be for another PR but we need to make some decisions about attaching user ids to these requests. We have existing functionality to access the users credentials on desktop but not on web. |
This also doesn't contain any notification content to the user about success/failure (apart from invalid submissions) and the response is unused. We need to think about what we want to return to users. |
In 46bae5d we use the existing username to build the request. This only works for desktop so we disabled the option for web in f61ded5 |
As of right now, there are 5 possible statuses for each of the files submitted in the request--I'll give suggestions for what to immediately show users based on each
For files that haven't been recently sent to the cache endpoint, state can be inferred from the following field combinations:
Note: the fields I just described above haven't yet been implemented in the ETL, I'm working on that now. The status response from the |
feature/refactor-environment
Description
The purpose of this PR is to add in the functionality for submitting the cache request and resolve #291. This PR does three main things:
FileDownloadServiceBaseURl
which was unused toAicsLoadBalancerBaseURl
Request Format
Testing
This pr has been tested against staging, but has not been tested against production as it is blocked by CORS policy.