Skip to content
This repository has been archived by the owner. It is now read-only.

Inline CSV report creation #137

Merged
merged 1 commit into from Sep 12, 2017

Conversation

Projects
None yet
2 participants
@c-w
Copy link
Member

commented Sep 12, 2017

Due to how the frontend project is set up, this approach will be easier to consume from the frontend than making a separate graphql call since we can just toggle the csv include via flux.

@c-w c-w requested a review from jcjimenez Sep 12, 2017

@c-w c-w force-pushed the inline-csv branch from 6effbe3 to 321e088 Sep 12, 2017

@jcjimenez
Copy link
Contributor

left a comment

LGTM

@c-w c-w referenced this pull request Sep 12, 2017

Merged

Wire-up CSV exporter #40

Inline CSV report creation
Due to how the frontend project is set up, this approach will be easier
to consume from the frontend than making a separate graphql call since
we can just toggle the csv include via flux.

@c-w c-w force-pushed the inline-csv branch from 321e088 to 02be015 Sep 12, 2017

@c-w c-w merged commit 61e9cc1 into master Sep 12, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@c-w c-w deleted the inline-csv branch Sep 12, 2017

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
You can’t perform that action at this time.