Skip to content

Refactor connection#16

Merged
abreckner merged 4 commits intomasterfrom
refactor-connection
Apr 28, 2020
Merged

Refactor connection#16
abreckner merged 4 commits intomasterfrom
refactor-connection

Conversation

@abreckner
Copy link
Contributor

Even though this class will change when we add Oauth and update the endpoint (it will be static instead of being passed in) the basics of get and post should act the same way.

@abreckner abreckner merged commit a4bbf66 into master Apr 28, 2020
Copy link
Contributor

@bedrock-adam bedrock-adam left a comment

Choose a reason for hiding this comment

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

👍

@abreckner abreckner deleted the refactor-connection branch May 1, 2020 05:14
lankz pushed a commit that referenced this pull request Jun 10, 2025
* Switch `get` tests from mocks to VCR

* Remove encoded auth

* refactor get and post to be more similar and add content type to post

* rubocop fix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants