Skip to content
This repository has been archived by the owner on Jun 27, 2019. It is now read-only.

Escape line breaks when generating package.json #226

Merged
merged 2 commits into from
Jan 17, 2018
Merged

Escape line breaks when generating package.json #226

merged 2 commits into from
Jan 17, 2018

Conversation

bradleypriest
Copy link
Contributor

Running zapier convert for an existing Zapier app with a multiline description generated an invalid JSON in package.json which then causes the whole process to fail with something similar to

Error! Unexpected token
 in JSON at position 149

Running `zapier convert` for an existing Zapier app with a multiline description generated an invalid JSON in package.json which then causes the whole process to fail with something similar to 

```
Error! Unexpected token
 in JSON at position 149
```
@xavdid xavdid requested a review from eliangcs January 15, 2018 21:16
@xavdid
Copy link
Contributor

xavdid commented Jan 15, 2018

Thanks for the contribution @bradleypriest!

We'll need you to sign the CLA (@bcooksey is that bot around?) but once your changes are approved by the engineer who has been working on the converter, you should be good to go!

@bcooksey
Copy link
Contributor

Kicked it off. Should be able to click the details link in the checks section now @bradleypriest

Copy link
Member

@eliangcs eliangcs left a comment

Choose a reason for hiding this comment

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

I didn't pull it since the change looks simple enough. Thanks for the fix!

@eliangcs eliangcs merged commit b47513f into zapier:master Jan 17, 2018
@bradleypriest bradleypriest deleted the patch-1 branch January 17, 2018 05:04
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.

4 participants