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

Added a clear example of the formatting #108

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

aop
Copy link

@aop aop commented Aug 13, 2022

Changed the method to describe the fact that the HTTP method should be GET (even though the non-standard SET seems to work as well).

Also added an example clearing out how the payload should be input.

Changed the method to describe the fact that the HTTP method should be GET (even though the non-standard SET seems to work as well).
Also added an example clearing out how the payload should be input.

Update go-eCharger API v1 EN.md
@@ -199,6 +199,10 @@ For all parameters that can be set, the format is for the command:
| WiFi local network | http://x.x.x.x/mqtt?payload= |
| Cloud: REST Api | https://api.go-e.co/api?token=TOKEN&payload=MESSAGE |

For example setting the Ampere value for the PWM signaling to the value of "8" would be a

```GET http://192.168.4.1/mqtt?payload=amp=8```
Copy link

Choose a reason for hiding this comment

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

This doesn't look like valid query string. Is the second (non URL encoded) = correct?

Copy link
Author

Choose a reason for hiding this comment

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

You're probably right. I was using Postman to test out the request and that worked just fine so I'd assume that some encoding is happening in the background, or then this just works for some reason.

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