Add catch blocks (!9) · Merge requests · GitLab.org / trello-power-up · GitLab

Snippets Groups Projects

Do not update/delete: Banner broadcast message test data

Do not update/delete: Notification broadcast message test data

Merged username-removed-408677 requested to merge add-catch into master 7 years ago

All threads resolved!

Adds the error handling provided by the Trello API. I reached out to the team that supports the power up API and it looks like that error handling only logs to the console. We may want to consider adding something to give a visual error message to our users but this is good enough for now.

Closes #19 (closed)

Edited 7 years ago by username-removed-408677

Activity

username-removed-408677 assigned to @filipa 7 years ago

assigned to @filipa
username-removed-408677 changed the description 7 years ago

changed the description
username-removed-408677 mentioned in issue #28 7 years ago

mentioned in issue #28
username-removed-408677 @ClemMakesApps · 7 years ago

Author Maintainer

Created https://gitlab.com/gitlab-org/trello-power-up/issues/28 for the proposal of adding visual error messages
Filipa Lacerda @filipa started a thread on an old version of the diff 7 years ago

Resolved 7 years ago by Filipa Lacerda
Last reply by Filipa Lacerda 7 years ago

Filipa Lacerda approved this merge request 7 years ago

approved this merge request

Filipa Lacerda resolved all discussions 7 years ago

resolved all discussions

Filipa Lacerda merged 7 years ago

merged

username-removed-408677 changed milestone to 7 years ago

changed milestone to %1.0

Please register or sign in to reply