Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

User with notification disabled still receive the notifications #12596

Closed
mattiagiupponi opened this issue Sep 20, 2024 · 0 comments · Fixed by #12597
Closed

User with notification disabled still receive the notifications #12596

mattiagiupponi opened this issue Sep 20, 2024 · 0 comments · Fixed by #12597
Assignees
Milestone

Comments

@mattiagiupponi
Copy link
Contributor

Expected Behavior

If the notification in the user panel are disabled, the user should not receive the notification

Actual Behavior

The user still get the notification

Steps to Reproduce the Problem

  1. enable notification system in GeoNode
  2. access to the personal notification panel
  3. disable all notification
  4. upload something

Specifications

  • GeoNode version: master/44x/43x
  • Installation type (vanilla, geonode-project): any
  • Installation method (manual, docker): any
  • Platform:
  • Additional details:
@mattiagiupponi mattiagiupponi added this to the 4.4.0 milestone Sep 20, 2024
@mattiagiupponi mattiagiupponi self-assigned this Sep 20, 2024
github-actions bot pushed a commit that referenced this issue Sep 20, 2024
#12597)

* [Fixes #12596] User with notification disabled still receive the notifications

* [Fixes #12596] User with notification disabled still receive the notifications

(cherry picked from commit 3973941)
mattiagiupponi added a commit that referenced this issue Sep 20, 2024
#12597) (#12598)

* [Fixes #12596] User with notification disabled still receive the notifications

* [Fixes #12596] User with notification disabled still receive the notifications

(cherry picked from commit 3973941)

Co-authored-by: mattiagiupponi <51856725+mattiagiupponi@users.noreply.github.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant