Skip to content

Replace empty .then() with .catch() and error logging #1100

Closed
@MaxSchaefer

Description

@MaxSchaefer

Describe the chore

We have many empty .then()'s all around the place. We did this to satisfy our linter but we should use .catch() with error logging instead because an empty .then() omits every error that might be thrown.

  • Disallow empty .then() via the linter
  • Replace empty .then() with .catch() and error logging

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions