-
Notifications
You must be signed in to change notification settings - Fork 71.9k
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
Test new dev version of minimed-connect-to-nightscout #6657
Conversation
Try out plugin with recent fixes, thanks to @FredMK.
nightscout/minimed-connect-to-nightscout#17 @Sulka, and @FredMK here is a patch to test Nightscout against Frigyes' recent changes. Been thinking a lot about the kind of observability needed to verify the rate limiting in features like this and am hopeful to have some ideas to help solve this kind of thing across plugins. |
@psonnera, @ffernandezro, if this tests ok, I will prep the mmconnect plugin and cgm-remote-monitor for release. |
Right - @bewest I'd love to get the admin messages feature out for release after next, so maybe we should think of extending the plugin API to 1) include a clean method of triggering data loads using the event bus and a defined API for storing new CGM entries and treatments, and 2) method for reporting the integration status, so we'd have a channel to expose issues to users. Would mean additional work for mmconnect but this would clean the system a lot |
I need help for testing because i cannot find a way to manage with Carelink connection MiniMed Connect error: Error: Request failed with status code 403stack: Error: Request failed with status code 403 |
@tudorfarcas2002 I believe this problem is discussed in at least #6616 (comment) (point 4). nightscout/minimed-connect-to-nightscout#15 might also be relevant. |
Thank you for testing, @tudorfarcas2002. Which country are you in? I have found that sometimes setting country code makes a difference, eg |
For anyone using this plugin, if it's not working for you currently, please remove |
Hello. |
Update to version available in npm.
…te-monitor into wip/bewest/mmconnect
I have deployed the new dev version in Heroku, but I am still receiving HTTP 403 (EU): at async doLoginEu1 (/app/node_modules/minimed-connect-to-nightscout/carelink.js:147:16) MMConnect as a standalon node.js app is working fine for many users, but when I try to run it in Heroku it receives HTTP 403. Are you sure, that Medtronic didn't blacklisted Heroku? |
Same here. Tried both with an US based and Europe based app. Same issue. Using latest cgm-remote-monitor dev version |
Great, thanks for testing! If it's not working in heroku, please ensure |
Nightscout it's working with Carelink. Thanks Ben West , Sulka and all developers |
Hi all ... I've read this post and I can't undestands if it apply also to Minimed 780G or no yet ??. Thanks all !!! |
Try out plugin with recent fixes, thanks to @FredMK. If this tests well we will promote minimed-connect-to-nightscout changes to master and do full release cycle there and update this thread after.