Skip to content

adafruit_io#unsubscribe(group_key=xxx) should unsubscribe thegroup ke… #88

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

Merged
merged 1 commit into from
Apr 11, 2022

Conversation

awgrover
Copy link
Contributor

fixes #87

I have not tested this. I only noted the inconsistency of feed_key vs group_key.

I would like this fix to go in the adafruit-circuitpython-bundle-7.x-mpy, please advise if I need to submit another PR, and against which tag/branch.

@FoamyGuy
Copy link
Contributor

Thanks for the fix @awgrover. There is nothing further to do for the bundle. Once merged, this will automatically make it's way into the bundle which gets generated automatically each night when there are changes to any library contained within it (new version most nights)

@FoamyGuy FoamyGuy merged commit 20e4884 into adafruit:main Apr 11, 2022
evaherrada added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Apr 11, 2022
Updating https://github.com/adafruit/Adafruit_CircuitPython_EMC2101 to 1.1.12 from 1.1.11:
  > Merge pull request adafruit/Adafruit_CircuitPython_EMC2101#22 from rivimey/convrate_docstring
  > Update Black to latest.
  > Fixed readthedocs build
  > Consolidate Documentation sections of README

Updating https://github.com/adafruit/Adafruit_CircuitPython_AdafruitIO to 5.6.5 from 5.6.4:
  > Merge pull request adafruit/Adafruit_CircuitPython_AdafruitIO#88 from awgrover/unsubscribe-group-key-87
  > Update Black to latest.

Updating https://github.com/adafruit/Adafruit_CircuitPython_BLE_Apple_Notification_Center to 0.10.0 from 0.9.7:
  > Merge pull request adafruit/Adafruit_CircuitPython_BLE_Apple_Notification_Center#16 from todbot/add_actions
  > Update Black to latest.
  > Fixed readthedocs build
  > Consolidate Documentation sections of README
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.

IO_MQTT.unsubscribe doesn't unsubscribe group-key? (adafruit_io.py)
3 participants