Skip to content
This repository has been archived by the owner on Aug 9, 2022. It is now read-only.

Added Toasts for all API actions #116

Merged

Conversation

davidcui1225
Copy link
Contributor

Issue #, if available:
N/A
Description of changes:
Added toast notifications for all API actions:

  • Reporting Homepage
    • Error toast when an on-demand download fails
    • Success toast when an on-demand download succeeds
    • Error toast when the Reports table content cannot be loaded correctly
    • Error toast when the Report definitions table content cannot be loaded correctly
  • Edit report definition
    • Error toast when updating a report definition fails
  • Report details
    • Error toast when getting all report details fails
  • Report definition details
    • Error toast when getting all report definition details fails
    • On Demand Definitions:
      • Error toast when generating a report from the Report definition details page fails
      • Success toast when generating a report succeeds
    • Schedule:
      • Error toast when disabling/enabling a scheduled definition returns error
      • Success toast when disabling/enabling a scheduled definition succeeds

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@davidcui1225 davidcui1225 merged commit 55f8e8b into opendistro-for-elasticsearch:dev Oct 8, 2020
@davidcui1225 davidcui1225 deleted the toast-notifs branch October 26, 2020 16:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants