You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Ive found that you have a Group table which can be appended to with a group name. However there is no post method or view for adding a group. Sure it can be done manually. Also, if we have two entries one for a user who is authenticated and the same user being in a group. The send_group_notification sends two notifications. One for the specific user and another for the group which also goes to the same user. This is not correct as you would only want one notification to be sent. This is happening here in save_info method of webpush views.py
if request.user.is_authenticated or group_name:
# Save the subscription info with subscription data
# as the subscription data is a dictionary and its valid
subscription = subscription_form.get_or_save()
web_push_form.save_or_delete(
subscription=subscription, **user=request.user,**
status_type=status_type, group_name=group_name)
As both group and user information is sent while it should be exclusive one or the other.
The documentation for group creation is sparse and not clear.
The text was updated successfully, but these errors were encountered:
Ive found that you have a Group table which can be appended to with a group name. However there is no post method or view for adding a group. Sure it can be done manually. Also, if we have two entries one for a user who is authenticated and the same user being in a group. The send_group_notification sends two notifications. One for the specific user and another for the group which also goes to the same user. This is not correct as you would only want one notification to be sent. This is happening here in save_info method of webpush views.py
As both group and user information is sent while it should be exclusive one or the other.
The documentation for group creation is sparse and not clear.
The text was updated successfully, but these errors were encountered: