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

Add return_path param to send() docs #169

Merged
merged 4 commits into from May 18, 2018
Merged

Conversation

jgzamora
Copy link
Contributor

@jgzamora jgzamora commented May 17, 2018

Fixing missing param documentation raised in #161. Also updated some of the other docs that had small errors.

  • Add retrun_path
  • Check if other params are missing and fix errors

Copy link
Contributor Author

@jgzamora jgzamora left a comment

Choose a reason for hiding this comment

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

Some outstanding stuff

used
:param bool use_draft_template: Default to False. Set to true if you
:param str template: ID of template to be used. Setting a template
overrides set the HTML and text params
Copy link
Contributor Author

Choose a reason for hiding this comment

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

remove "set"

if dict: ``{'address': {'name': 'Name', 'email': 'me' }}``
:param str recipient_list: ID of recipient list. If this is set,
the `recipients`
param will be ignored
Copy link
Contributor Author

Choose a reason for hiding this comment

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

unnecessary extra line

:param str description: Description of transmission
:param str campaign: Campaign of transmission
:param dict metadata: Any data you want to send along with
transmission, used in WebHooks
:param dict substitution_data: Corresponds to substitutions in
html/text content. See `substitutions reference
<https://developers.sparkpost.com/api/substitutions-reference.html>`_.
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 did this because of line length linting, but it's smart enough to ignore links. Link works fine though

@jgzamora jgzamora self-assigned this May 18, 2018
@jgzamora jgzamora added the docs label May 18, 2018
avigoldman
avigoldman previously approved these changes May 18, 2018
@jgzamora jgzamora merged commit 351e9ae into master May 18, 2018
@jgzamora jgzamora deleted the document-missing-params branch May 18, 2018 19:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants