Invalid group name about django-nyt HOT 4 CLOSED

The group is supposed to contain the notification key such that people can subscribe to only specific channels (notifications).

That should be fine, however there could be an issue with your notification key - could we somehow allow notification keys that don't conform to only alphanumerics, hyphens, or periods. ?

from django-nyt.

The problem is the second Group(...).discard function:
Group(settings.NOTIFICATION_CHANNEL).discard(message.reply_channel), because it uses non formated settings.NOTIFICATION_CHANNEL (value is: "nyt_all-{notification_key:s}") which contains non allowed characters. I also think it was left there by mistake

from django-nyt.

I totally missed the point in this issue, thanks @ztomaz :)

from django-nyt.

Thanks @benjaoming, thanks @ztomaz :)

from django-nyt.