Skip to content

Conversation

dvaldivia
Copy link
Collaborator

@dvaldivia dvaldivia commented Jul 14, 2021

Introduces native browser file download behavior

Screen Shot 2021-07-14 at 12 54 12 PM

Test this using only the go server, since the react server won't proxy the request properly

Signed-off-by: Daniel Valdivia 18384552+dvaldivia@users.noreply.github.com

Copy link
Collaborator

@bexsoft bexsoft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure if I have something missing, I downloaded the new code, restarted my server & frontend but when I try to download any file the page reloads and redirects me to dashboard. Not sure if some route is missing in endpoints.go:

2021-07-15 13-41-39 2021-07-15 13_43_11

@dvaldivia
Copy link
Collaborator Author

@bexsoft this feature doesn't work through the React development server, try this using the final go server with the commited assets on port 9090

@bexsoft
Copy link
Collaborator

bexsoft commented Jul 15, 2021

@bexsoft this feature doesn't work through the React development server, try this using the final go server with the commited assets on port 9090

I see, please add this information to the PR to avoid confusion

@dvaldivia
Copy link
Collaborator Author

@bexsoft done

Copy link
Collaborator

@bexsoft bexsoft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please reset the state of downloadingFiles reducer as it keeps in loading state even when download has already completed

2021-07-15 14-09-06 2021-07-15 14_14_32

Signed-off-by: Daniel Valdivia <18384552+dvaldivia@users.noreply.github.com>
@dvaldivia
Copy link
Collaborator Author

@bexsoft done

@harshavardhana harshavardhana merged commit 27489d9 into minio:master Jul 15, 2021
@harshavardhana harshavardhana deleted the file-download branch July 15, 2021 20:26
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.

3 participants