Skip to content

Add Client::set_headers with the possibility to set additional custom…#17

Closed
tolvstein wants to merge 1 commit into
QuickPay:masterfrom
tolvstein:master
Closed

Add Client::set_headers with the possibility to set additional custom…#17
tolvstein wants to merge 1 commit into
QuickPay:masterfrom
tolvstein:master

Conversation

@tolvstein
Copy link
Copy Markdown
Contributor

… headers. Used in Request::execute to possibly set the Content-Length of the JSON request body when sending form data

Also added the header Content-Type to align the request headers with the requirements of the QuickPay API docs

… headers. Used in Request::execute to possibly set the Content-Length of the JSON request body when sending form data

Also added the header Content-Type to align the request headers with the requirements of the QuickPay API docs
@lsolesen
Copy link
Copy Markdown
Contributor

@tolvstein Could you please update your pull request and base it off the latest code?

@lsolesen
Copy link
Copy Markdown
Contributor

@PerfectSolutionApS Could you update the pull request based on latest master - or should it just be closed?

@Sthing
Copy link
Copy Markdown
Collaborator

Sthing commented Dec 16, 2019

Hi @PerfectSolutionApS

Release 1.1.0 adds support for custom headers.

Is the other part of your PR still relevant?

/Søren Thing

@tolvstein
Copy link
Copy Markdown
Contributor Author

It is no longer relevant.

@tolvstein tolvstein closed this Dec 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants