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

Allow cli access to request #298

Merged
merged 3 commits into from
Apr 16, 2018
Merged

Conversation

buildmaster
Copy link
Contributor

Adding documentation of cleanedRequest and rawRequest as created in PR zapier/zapier#16747

exposed in hooks and after zapier/zapier#16747 it'll be exposed in the `getAccessToken` method
README-source.md Outdated
querystring:'key=value;key2=value2',
content:'{key:value}'
}
```
Copy link
Member

@bryanhelmig bryanhelmig Apr 13, 2018

Choose a reason for hiding this comment

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

Since these are Javascript objects, I'd do something more complete and neater like:

{
  method: 'POST',
  querystring: 'foo=bar&baz=qux',
  headers: {
    'Content-Type': 'application/json'
  },
  content: '{"hello": "world"}'
}

Copy link
Contributor Author

Choose a reason for hiding this comment

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

lazy me. Will fix

README-source.md Outdated
key: 'value'
}
}
```
Copy link
Member

Choose a reason for hiding this comment

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

As well as a matching cleaned version:

{
  method: 'POST',
  querystring: {
    foo: 'bar',
    baz: 'qux'
  },
  headers: {
    'Content-Type': 'application/json'
  },
  content: {
    hello: 'world'
  }
}

README-source.md Outdated
@@ -633,6 +633,41 @@ module.exports = {
};
```

### `bundle.rawRequest`
***`bundle.rawRequest` is only available in the `perform` for web hooks and `getAccessToken` for oauth authentication methods***
Copy link
Member

Choose a reason for hiding this comment

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

Might do > blockquote? Just a nit, don't really mind either way. /shrug

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 agree > looks better. I'd copied the formatting from the end of the bundle meta. So I've changed that too

@buildmaster buildmaster self-assigned this Apr 16, 2018
@buildmaster buildmaster merged commit 9c6225c into master Apr 16, 2018
@buildmaster buildmaster deleted the allow-cli-access-to-request branch April 16, 2018 23:22
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