Skip to content

Commit

Permalink
fix: No need to show toast messages on success
Browse files Browse the repository at this point in the history
Signed-off-by: Julius Härtl <jus@bitgrid.net>
  • Loading branch information
juliusknorr committed Aug 16, 2023
1 parent 5e2aead commit 8eaaa67
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions src/modules/sidebar/mixins/shareAPI.js
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,6 @@ export default {
}
try {
const res = await axios.post(generateUrl('/apps/tables/share'), data)

Check failure on line 40 in src/modules/sidebar/mixins/shareAPI.js

View workflow job for this annotation

GitHub Actions / eslint

'res' is assigned a value but never used
showSuccess(t('tables', 'Saved new share with "{userName}".', { userName: res.data.receiverDisplayName }))
} catch (e) {
displayError(e, t('tables', 'Could not create share.'))
return false
Expand All @@ -50,7 +49,6 @@ export default {
async removeShareFromBE(shareId) {
try {
await axios.delete(generateUrl('/apps/tables/share/' + shareId))
showSuccess(t('tables', 'Share was deleted'))
} catch (e) {
displayError(e, t('tables', 'Could not remove share.'))
}
Expand All @@ -59,7 +57,6 @@ export default {
async updateShareToBE(shareId, data) {
try {
await axios.put(generateUrl('/apps/tables/share/' + shareId + '/permission'), data)
showSuccess(t('tables', 'Share permission was updated'))
} catch (e) {
displayError(e, t('tables', 'Could not update share.'))
}
Expand Down

0 comments on commit 8eaaa67

Please sign in to comment.