StackStorm-Exchange / stackstorm-email

E-Mail Actions/Sensors for StackStorm
https://exchange.stackstorm.org/
Apache License 2.0
9 stars 45 forks source link

fix python3 issue in IMAPsensor #41

Closed mjsalmi closed 3 years ago

mjsalmi commented 3 years ago

Fixes #36 by simply converting uid to string. This change should be backwards compatible with python2 but I didn't really see the need to test it.

blag commented 3 years ago

Don't bother with Python 2 compatibility. We dropped support for that in all packs last month. Once you fix that this looks great.

blag commented 3 years ago

I forgot to mention - bump the version in pack.yaml by 0.0.1 and update the changelog with the new version and your change. Then I'll merge it.

blag commented 3 years ago

Awesome, thanks!