Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings
This repository was archived by the owner on Feb 28, 2023. It is now read-only.

Add support for idempotency_key and timestamp#17

Open
macobo wants to merge 1 commit intoheap:master from
macobo:timestamp-idempotency-key
Open

Add support for idempotency_key and timestamp #17
macobo wants to merge 1 commit intoheap:master from
macobo:timestamp-idempotency-key

Conversation

@macobo
Copy link
Contributor

@macobo macobo commented Apr 18, 2018

This continues work from #14 and also adds idempotency_key support.

@macobo macobo force-pushed the timestamp-idempotency-key branch 2 times, most recently from c397ec9 to 736f9bb Compare April 18, 2018 19:52
Copy link

coveralls commented Apr 18, 2018
edited
Loading

Coverage Status

Coverage remained the same at 100.0% when pulling 2fe3f82 on macobo:timestamp-idempotency-key into 0f0a353 on heap:master.

@macobo macobo force-pushed the timestamp-idempotency-key branch 3 times, most recently from d74acb9 to 7e77ea6 Compare April 18, 2018 20:05
@macobo macobo force-pushed the timestamp-idempotency-key branch from 7e77ea6 to 2fe3f82 Compare April 18, 2018 20:13
Copy link
Contributor Author

macobo commented Apr 18, 2018

I'm not sure about the failures here:

1.8 failure seems real, no support for this method, but others seem like things that would be failing on master as well.

Would it be worth dropping support for 1.8?

Adverbly reacted with thumbs up emoji

Copy link
Contributor

jeffling commented Apr 24, 2018
edited
Loading

<3

We could really use this as well :)

@drob @dorianj ? Think it would be possible to give one of us merge privileges?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /