Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: support strings option over HTTP API #203

Merged
merged 5 commits into from
Dec 2, 2020
Merged

Conversation

aschmahmann
Copy link
Contributor

@aschmahmann aschmahmann commented Nov 25, 2020

StringsOption was not properly supported over HTTP API before only over the CLI. This resolves that issue.

Note: Still needs tests as this was previously thought to be supported but was not.

@aschmahmann aschmahmann marked this pull request as ready for review November 30, 2020 19:15
@aschmahmann aschmahmann requested a review from lidel November 30, 2020 19:15
http/parse.go Outdated Show resolved Hide resolved
Copy link
Member

@Stebalien Stebalien left a comment

Choose a reason for hiding this comment

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

Some small nits but otherwise LGTM.

http/client.go Outdated Show resolved Hide resolved
http/parse.go Outdated Show resolved Hide resolved
http/parse.go Outdated Show resolved Hide resolved
http/parse.go Outdated Show resolved Hide resolved
request.go Outdated Show resolved Hide resolved
@aschmahmann aschmahmann requested a review from Stebalien December 2, 2020 05:41
@aschmahmann aschmahmann merged commit edc78ef into master Dec 2, 2020
@aschmahmann aschmahmann mentioned this pull request Feb 18, 2021
73 tasks
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.

2 participants