Skip to content
This repository was archived by the owner on Apr 5, 2018. It is now read-only.

Emailable export#44

Open
JoshStaff wants to merge 1 commit intoboboldehampsink:masterfrom
JoshStaff:master
Open

Emailable export#44
JoshStaff wants to merge 1 commit intoboboldehampsink:masterfrom
JoshStaff:master

Conversation

@JoshStaff
Copy link
Copy Markdown

@JoshStaff JoshStaff commented Oct 25, 2017

Able to select options on whether or not you want to export and email it later or download the response – This solved the issue with the export crashing prior to it presenting the result. Is this PR something you would consider beneficial? If so, I am happy to tidy it up (There are code changes / refactors I'd like to do), add tests & work with you on this.

… the result later or download the responsr
@boboldehampsink
Copy link
Copy Markdown
Owner

boboldehampsink commented Oct 26, 2017

This is on the map, yes. Can you tidy it up first? It's a mess (merge conflicts, checked-in csv's, no PSR-2, etc.)

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.

2 participants