fix: create preferences for all types instead of core#36984
fix: create preferences for all types instead of core#36984muhammadadeeltajamul merged 1 commit intomasterfrom
Conversation
| Creates missing account level preferences for notifications | ||
| """ | ||
| preferences = list(preferences) | ||
| notification_types = list(set( |
There was a problem hiding this comment.
This is a much simpler way to find and create missing preferences . may be we we can extract this code in another function if needed.
https://github.com/openedx/edx-platform/pull/36957/files#diff-ff9c6979d67deaf0053d33346b0f3eb30ab3ec9ae530227826ce685fa2a9e635R659
There was a problem hiding this comment.
We have complicated imports in notifications app. It will create circular import problem
|
2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production. |
|
2U Release Notice: This PR has been deployed to the edX production environment. |
1 similar comment
|
2U Release Notice: This PR has been deployed to the edX production environment. |
|
2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production. |
|
2U Release Notice: This PR has been deployed to the edX production environment. |
1 similar comment
|
2U Release Notice: This PR has been deployed to the edX production environment. |
3 participants
Create preferences for all notification types instead of core