Skip to content
This repository has been archived by the owner on Apr 14, 2024. It is now read-only.

feat: Added Stream Functionality #20

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

zymawy
Copy link

@zymawy zymawy commented Dec 11, 2020

A similar PR may already be submitted!
Please search among the pull requests before creating one.

Thanks for submitting a pull request! Please provide enough information so that others can review your pull request:
Summary

I run into in satiation where I just want to stream the merged files directly to the client. instead of saving it or forcing the client to save the merged file into their machine. by this PR, I just added the functionality the make response serve as
streamed file

For more information, see the CONTRIBUTING guide.

This PR fixes/implements the following bugs/features

  • [ x ] Stream File Into Browser

Explain the motivation for making this change. What existing problem does the pull request solve?

Test plan (required)

Demonstrate the code is solid. Example: The exact commands you ran and their output.

Closing issues

Fixes #

Checklist

  • Have you followed the guidelines in our CONTRIBUTING guide?
  • Have you lint your code locally prior to submission (make lint)?
  • Have you written new tests for your core changes, as applicable?
  • Have you successfully ran tests with your changes locally (make tests)?
  • I have squashed any insignificant commits
  • This change has comments for package types, values, functions, and non-obvious lines of code

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

Successfully merging this pull request may close these issues.

1 participant