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

PROGMEM for newer Arduino IDE releases #16

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

davedarko
Copy link

Hi, just stumbled over your API and tried to make it work. The main problem was that the way to store the strings in the program memory didn't work with newer Arduino IDEs anymore. Also the pusher link has changed. While debugging I renamed the getStringTableItem function of the WebSocketClient but didn't reset it. Other than that it now works with an ethernet feather wing and a teensy++ 2!

switched to eu server
updated to correct PROGMEM calls
renamed while debugging
@LaughingOakFarm
Copy link

Please merge this PR in! It works great and is needed to have this Lib continued to be used.

@01giraffe
Copy link

Hi,
I can't get it to connect from arduino to pusher.
Will you help me, maybe you can find what I am doing wrong.

The situation is as follows

I have added all keys in the pusherclient.cpp. between the "" as you had described.

In the example script in IDE, I haven't changed anything, but when I compile I get the following message: Compilation error: no matching function for call to 'PusherClient::connect(const char [18])'

When I remove the "your-api-key-here" in the line:
if(client.connect("your-api-key-here")) {

I no longer get compile errors. but get as response in the serial monitor:
21:44:16.328 -> Connecting to host.
21:44:16.328 -> Connection to host failed.
21:44:16.361 -> Connection to Pusher failed.

I suspect the error is in that if(client.connect line.

i hope you can help me to solve it.

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