krikkert/habitica
1
0
Fork 0
mirror of https://github.com/HabitRPG/habitica.git synced 2025-12-17 22:57:21 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix #10756: do not show push notification settings for email only notifications

Browse Source
This commit is contained in:
Matteo Pagliazzi
2018-10-18 14:21:35 +02:00
parent 1f44444a50
commit ffba435923
1 changed files with 9 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
website/client/components/settings/notifications.vue
View File

@@ -32,7 +32,7 @@
td td
input(type='checkbox', v-model='user.preferences.emailNotifications[notification]', input(type='checkbox', v-model='user.preferences.emailNotifications[notification]',
@change='set("emailNotifications", notification)') @change='set("emailNotifications", notification)')
td td(v-if="onlyEmailsIds.indexOf(notification) === -1")
input(type='checkbox', v-model='user.preferences.pushNotifications[notification]', input(type='checkbox', v-model='user.preferences.pushNotifications[notification]',
@change='set("pushNotifications", notification)') @change='set("pushNotifications", notification)')
hr hr
@@ -46,7 +46,7 @@ export default {
mixins: [notificationsMixin], mixins: [notificationsMixin],
data () { data () {
return { return {
notificationsIds: [ notificationsIds: Object.freeze([
'newPM', 'newPM',
'wonChallenge', 'wonChallenge',
'giftedGems', 'giftedGems',
@@ -59,7 +59,13 @@ export default {
'importantAnnouncements', 'importantAnnouncements',
'weeklyRecaps', 'weeklyRecaps',
'onboarding', 'onboarding',
], ]),
// list of email-only notifications
onlyEmailsIds: Object.freeze([
'importantAnnouncements',
'weeklyRecaps',
'onboarding',
]),
}; };
}, },
computed: { computed: {