Error handling #5

Merged
rozetko merged 4 commits from error-handling into master 2 years ago
rozetko commented 2 years ago
Owner

Changes in types:

  • rename TaskTableRowConfig type to ExportTask
  • rename QueryTableRowConfig type to DashboardQuery
  • new ExportStatus: Error
  • new errorMessage field in ExportTask

Changes:

  • use uuid for a task filename instead of using timestamp and datasource id
  • on export error, update status with error message and stop exporting
  • use multiple queries in a task instead of multiple tasks
  • re-organize ExportTask type

Other changes:

  • move types/target.ts to models/
  • move everything else from types/ directory into a single types.ts file
  • Exporter: prefix private methods names with _
Changes in types: - rename `TaskTableRowConfig` type to `ExportTask` - rename `QueryTableRowConfig` type to `DashboardQuery` - new `ExportStatus`: Error - new `errorMessage` field in `ExportTask` Changes: - use `uuid` for a task filename instead of using timestamp and datasource id - on export error, update status with error message and stop exporting - use multiple queries in a task instead of multiple tasks - re-organize `ExportTask` type Other changes: - move `types/target.ts` to `models/` - move everything else from `types/` directory into a single `types.ts` file - `Exporter`: prefix private methods names with `_`
rozetko added 3 commits 2 years ago
rozetko added 1 commit 2 years ago
rozetko merged commit bba98e4501 into master 2 years ago
rozetko deleted branch error-handling 2 years ago
The pull request has been merged as bba98e4501.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.