Skip to content

Using baseName in place of paramName in request header. Closes #3502#3504

Merged
wing328 merged 1 commit intoswagger-api:masterfrom
himanshu-mishra:issue-3502
Aug 3, 2016
Merged

Using baseName in place of paramName in request header. Closes #3502#3504
wing328 merged 1 commit intoswagger-api:masterfrom
himanshu-mishra:issue-3502

Conversation

@himanshu-mishra
Copy link
Copy Markdown
Contributor

No description provided.

@wing328
Copy link
Copy Markdown
Contributor

wing328 commented Aug 2, 2016

Thanks for the PR but your commit (as shown in the Commits tab) is not linked to your Github account, which means this PR won't count as your contribution in https://github.com/swagger-api/swagger-codegen/graphs/contributors.

Let me know if you need help fixing it.

Ref: https://github.com/swagger-api/swagger-codegen/wiki/FAQ#how-can-i-update-commits-that-are-not-linked-to-my-github-account

@himanshu-mishra
Copy link
Copy Markdown
Contributor Author

@wing328 Thanks for letting me know. Have verified my secondary email id and now the commit is linked to my account.

@wing328 wing328 merged commit 5103045 into swagger-api:master Aug 3, 2016
@wing328
Copy link
Copy Markdown
Contributor

wing328 commented Aug 3, 2016

@himanshu-mishra thanks for the contribution.

I've reviewed the api.mustache and is it correct to say that the Feign client does not support form parameters at the moment?

@himanshu-mishra
Copy link
Copy Markdown
Contributor Author

@wing328 I will verify and let you know

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