search

jazzband / django-push-notifications

Send push notifications to mobile devices through GCM or APNS in Django.
MIT License
2.24k stars 609 forks source link

Incompatible with Python 3.10 #622

Open bertonha opened 2 years ago

bertonha commented 2 years ago

This package is not supported on Python 3.10.

Mainly because of the sub dependency hyper (discontinued) is importing Iterable from collection instead of collection.abc

hyper is dependency of apns2

realsama commented 2 years ago

I am experiencing this issue also.

ImportError: cannot import name 'Iterable' from 'collections'

koedal commented 2 years ago

I also have this issue. Are there any plans to support Python 3.10?

andrewsanchez commented 1 year ago

Quick fix is to patch it like so:

https://github.com/healthvana/h2/commit/d67c6ca10eb7f79c0737c37fdecfe651307a7414

samirsd commented 1 year ago

can we address this? it's been over a year. is there a plan to merge this patch or otherwise?

samirsd commented 1 year ago

Quick fix is to patch it like so:

healthvana/h2@d67c6ca

would you mind explaining how to do this? I'm assuming you'd start by forking django-push-notifications and whatever dependency is problematic and use that...? however I'm not sure where to find the hyper or apns2 packages in this project.

merwok commented 1 year ago

No, that link shows a monkey-patch, meaning changing a module at runtime. It is not a source patch to be applied to source code.

shree-ranga commented 1 year ago

Is this not fixed yet? I'm still facing module not found 'apns2'.

harikvpy commented 9 months ago

+1

koedal commented 9 months ago

Any updates here?

ethanitovitch commented 7 months ago

Still seeing this issue on python 3.11

kmasuhr commented 6 months ago

+1

mirodil1 commented 5 months ago

+1

ssyberg commented 4 months ago

Still an issue!

SaravananRamanathan commented 3 months ago

I see that, this was opened on Oct 6, 2021, and it's still a issue until now, anyone know of the current state of things? what do we need to support python 3.10+ ?

qx commented 3 months ago

still bug! Is there anyone still maintaining this project?

ssyberg commented 3 months ago

I believe this is addressed in this PR that I think is almost ready for merge: https://github.com/jazzband/django-push-notifications/pull/696

qx commented 3 months ago

@ssyberg you can test by yourself, It easy to appear

ssyberg commented 3 months ago

@ssyberg you can test by yourself, It easy to appear

I don't know what you mean

qx commented 3 months ago

This is a prone issue and I don't think it's been fixed yet

ssyberg commented 3 months ago

This is a prone issue and I don't think it's been fixed yet

You mean you don't think that PR addresses it? As I mentioned it hasn't been merged yet.

coratype commented 1 month ago

So has this been fixed? I'm getting the Iterable thing

illymarev commented 1 week ago

As Python 3.8 is close to EOL, many people are migrating to newer versions. Are there any updates on this issue?