Skip to content
This repository was archived by the owner on Aug 27, 2021. It is now read-only.

Conversation

jovel
Copy link
Contributor

@jovel jovel commented Oct 1, 2015

Ticket: MAKO-3603

Matching apiary: jsapi

(edited by mbernier to remove URLS to resources)

Copy link
Contributor

Choose a reason for hiding this comment

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

Im assuming this is a unix timestamp. Do we normally specify that?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

So it's marked as timestamp in apiary but something in the docs compiler throws up when I write timestamp so now it's number.

Copy link
Contributor

Choose a reason for hiding this comment

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

When tested locally Im seeing a DELETE in the request url for this section. Not sure why though.

DELETE https://api.sendgrid.com/v3/v3/suppression/bounces/{email} HTTP/1.1

Copy link
Contributor

Choose a reason for hiding this comment

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

Nevermind. This section looks good now after pulling your latest changes

@dmowcomber
Copy link
Contributor

LGTM. Tested locally

@mbernier
Copy link
Contributor

mbernier commented Oct 4, 2015

@jovel @dmowcomber Is this ready to go live?

@jovel
Copy link
Contributor Author

jovel commented Oct 5, 2015

@mbernier Yup, this doc is ready to go live if you're ok with the wording. The actual endpoint has been up since makoapi was first in beta.

mbernier pushed a commit that referenced this pull request Oct 6, 2015
@mbernier mbernier merged commit 98acf36 into develop Oct 6, 2015
@mbernier mbernier deleted the mako-3603-v3-bounces-api branch October 6, 2015 19:59
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.

3 participants