Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

There is an API call to remove all reports at once #384

Open
mhdirkse opened this issue Apr 15, 2024 · 0 comments
Open

There is an API call to remove all reports at once #384

mhdirkse opened this issue Apr 15, 2024 · 0 comments

Comments

@mhdirkse
Copy link
Member

The Ladybug backend supports an API call now to remove all reports at once. All reports of the debug tab at once and another endpoint to remove all reports of the test tab at once. That backend call is useful for the Cypress tests. Should we also use it in the ladybug-frontend? I see the following options:

  • We make a separate button in the GUI te remove all reports; or
  • There is a checkbox to select all reports already. If the user clicks the existing delete button, then the frontend checks whether all reports are selected. If so, the delete all API call is used. Otherwise, each report is deleted separately. Drawback: if the frontend and backend are out of sink, too many reports may be deleted; or
  • Because of these considerations, ladybug-frontend should always delete each report separately. In that case the delete all API call should stay in the backend to support the Cypress tests.
@MatthijsSmets MatthijsSmets added this to To do in Ladybug frontend via automation May 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

No branches or pull requests

1 participant