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

Update docs to remove Digest references #323

Merged
merged 1 commit into from
May 29, 2018

Conversation

cvanderkolk
Copy link
Contributor

Quick doc update to remove references to Digest auth, which isn't supported yet directly in CLI. It is, however, still implementable using Custom auth.

Copy link
Contributor

@BrunoBernardino BrunoBernardino left a comment

Choose a reason for hiding this comment

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

Nice and clean! Thanks for tackling this!

@@ -362,7 +362,7 @@ <h2 id="build">build</h2>
<li>Generates and validates app definition.</li>
<li>Detects dependencies via browserify (optional)</li>
<li>Zips up all needed <code>.js</code> files. If you want to include more files, add a &quot;includeInBuild&quot; property (array with strings of regexp paths) to your <code>.zapierapprc</code>.</li>
<li>Moves the zip to <code>build/build.zip</code></li>
<li>Moves the zip to <code>build/build.zip</code> and <code>build/source.zip</code></li>
Copy link
Contributor

Choose a reason for hiding this comment

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

Ah, this was my fault, I forgot to run npm run docs when merging that PR!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I figured you had made some changes earlier! 👍

@cvanderkolk cvanderkolk merged commit baec38e into master May 29, 2018
@cvanderkolk cvanderkolk deleted the removing-digest-from-readme branch May 29, 2018 14:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants