amy/donetick-fe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request #8 from dkhalife/bugfix/handle-notification-target-failure

Browse source 
Bugfix: handle request error when updating notification type
This commit is contained in:
Mohamad Tarbin 2024-12-31 02:27:47 -05:00 • committed by GitHub
commit 1d0f9ed1ed
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 6 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/views/Settings/NotificationSetting.jsx
View file

@ -129,7 +129,11 @@ const NotificationSetting = () => {
target: chatID, target: chatID,
type: Number(notificationTarget), type: Number(notificationTarget),
}).then(resp => { }).then(resp => {
alert('Notification target updated') if (resp.status != 200) {
alert(`Error while updating notification target: ${resp.statusText}`)
return
}
setUserProfile({ setUserProfile({
...userProfile, ...userProfile,
notification_target: { notification_target: {
@ -137,6 +141,7 @@ const NotificationSetting = () => {
type: Number(notificationTarget), type: Number(notificationTarget),
}, },
}) })
alert('Notification target updated')
}) })
} }
return ( return (