Skip to content
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

Use csv-stringify to convert to CSV #72

Merged
merged 1 commit into from
Nov 13, 2018
Merged

Conversation

Sohalt
Copy link
Contributor

@Sohalt Sohalt commented Nov 13, 2018

this fixes escaping data for CSV export

this fixes escaping data for CSV export
@simonv3 simonv3 merged commit 0076e85 into simonv3:develop Nov 13, 2018
@simonv3
Copy link
Owner

simonv3 commented Nov 13, 2018

@Sohalt I don't have the capacity to release this to sandstorm (I've long since forgotten how too). Is there anything I can do to help you deploy this?

@Sohalt
Copy link
Contributor Author

Sohalt commented Nov 13, 2018

Hi, thanks for merging this so quickly. I'll see what it takes to get this released and get back to you, if I need anything.

@Sohalt
Copy link
Contributor Author

Sohalt commented Nov 14, 2018

Basically just follow this: https://docs.sandstorm.io/en/latest/developing/publishing-apps/

I don't know if the changes on the develop branch since the last release are stable. If so it would make sense to update the changelog accordingly and make a release. If not it might make sense to just cherrypick the csv fix.

I can't release the app myself because I don't have the signing key.

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.

2 participants