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

Update ApiLibs #50

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

Conversation

mjwsteenbergen
Copy link
Collaborator

@mjwsteenbergen mjwsteenbergen commented Feb 6, 2022

Hi! After wayy to long, here's my new update.

I've reworked the internals, so that it is easier for this library to use. I hope in the future to pull stuff out even further, so you will have even less dependencies (you don't even use Restsharp atm)

I could not get the UI to work, so that I leave to you.

Closes mjwsteenbergen/ApiLibs#33

@mjwsteenbergen
Copy link
Collaborator Author

Hi @dominicusmento. Any update on this

@dominicusmento
Copy link
Owner

Hi @dominicusmento. Any update on this

Hi! I've tried to use the tradfriUI project, main.cs, to generate the new secret from your master but I'm getting the "Request timed out" for it.
It seems that ConnectPSK function works fine, but then calling the tradfriController.GenerateAppSecret() just times out.
Calling it without ConnectPSK() previously, fails with null reference exception on _coapClient.
Can you take a look at it?

When using an existing appkey with app name etc. - unit test works fine.

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.

Package 1.61.0 is not latest package from master
2 participants