Skip to content
This repository has been archived by the owner on Dec 2, 2022. It is now read-only.

Fix some wrong parameter names in WebAPI #178

Merged
merged 1 commit into from
Mar 20, 2020

Conversation

mtgto
Copy link
Contributor

@mtgto mtgto commented Mar 19, 2020

I found some parameters are mistaken.

sendMessage uses camelCase params, but it is snake_case, truly.
https://api.slack.com/methods/chat.postMessage

sendEphemeral also has mistakes.
https://api.slack.com/methods/chat.postEphemeral

fileInfo has bad arguments which are not matching api doc.
https://api.slack.com/methods/files.info

p.s. I only checked camelcase parameters. There might be more mistakes.

Verified

This commit was signed with the committer’s verified signature.
GuySartorelli Guy Sartorelli
@pvzig
Copy link
Owner

pvzig commented Mar 20, 2020

Thanks! 🙇

@pvzig pvzig merged commit 229e4ee into pvzig:master Mar 20, 2020
@mtgto mtgto deleted the fix_webapi_parameters branch March 21, 2020 00:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants