IntenseWebs/mattermost
3
0
Fork 0
mirror of https://github.com/mattermost/mattermost.git synced 2025-02-25 18:55:24 -06:00
Code Issues Packages Projects Releases Wiki Activity

MM-24386/MM-24465 Enable link previews and full push notifications by default (#14565)

Browse Source 
* MM-24386 Update default push notification contents to full

* MM-24465 Enable link previews by default

* Fix a unit test relying on the old default
This commit is contained in:
Harrison Healey
2020-05-20 09:23:13 -04:00
committed by GitHub
parent aad76a13e8
commit f62493145a
2 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
app/notification_push_test.go
View File

@@ -1296,6 +1296,7 @@ func TestAllPushNotifications(t *testing.T) {
defer pushServer.Close()
th.App.UpdateConfig(func(cfg *model.Config) {
*cfg.EmailSettings.PushNotificationContents = model.GENERIC_NOTIFICATION
*cfg.EmailSettings.PushNotificationServer = pushServer.URL
})

4
model/config.go
View File

@@ -369,7 +369,7 @@ func (s *ServiceSettings) SetDefaults(isUpdate bool) {
}
if s.EnableLinkPreviews == nil {
s.EnableLinkPreviews = NewBool(false)
s.EnableLinkPreviews = NewBool(true)
}
if s.EnableTesting == nil {
@@ -1459,7 +1459,7 @@ func (s *EmailSettings) SetDefaults(isUpdate bool) {
}
if s.PushNotificationContents == nil {
s.PushNotificationContents = NewString(GENERIC_NOTIFICATION)
s.PushNotificationContents = NewString(FULL_NOTIFICATION)
}
if s.EnableEmailBatching == nil {