Skip to content

docs: add documentation on the on-behalf-of header usage - #1104

Merged
childish-sambino merged 2 commits into
sendgrid:masterfrom
ayandyl:feature/doc-subuser-header
Apr 28, 2020
Merged

docs: add documentation on the on-behalf-of header usage#1104
childish-sambino merged 2 commits into
sendgrid:masterfrom
ayandyl:feature/doc-subuser-header

Conversation

@ayandyl

@ayandyl ayandyl commented Apr 25, 2020

Copy link
Copy Markdown
Contributor

Fixes

The use case of performing API calls on behalf of subusers is not documented.

While we're waiting on #600 to provide an interface to supply the subuser username,
this PR is a quickfix to at least document how to set the on-behalf-of header
to accomplish the use case.

Checklist

  • I acknowledge that all my contributions will be made under the project's license
  • I have made a material change to the repo (functionality, testing, spelling, grammar)
  • I have read the Contribution Guidelines and my PR follows them
  • I have titled the PR appropriately
  • I have updated my branch with the master branch
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation about the functionality in the appropriate .md file
  • I have added inline documentation to the code I modified

If you have questions, please file a support ticket, or create a GitHub Issue in this repository.

@thinkingserious thinkingserious added the status: code review request requesting a community code review or review from Twilio label Apr 25, 2020
@childish-sambino childish-sambino changed the title add documentation on the on-behalf-of header usage docs: add documentation on the on-behalf-of header usage Apr 28, 2020
@childish-sambino
childish-sambino merged commit fe835e1 into sendgrid:master Apr 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: code review request requesting a community code review or review from Twilio

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants