alan-turing-institute / DTBase

A starting point from which digital twins can be developed.
MIT License
11 stars 4 forks source link

112 editing deleting sensors #183

Closed GiorgioCerro closed 7 months ago

GiorgioCerro commented 7 months ago

There is an edit button for each sensor in the list. If clicked, a pop up window opens with a form for editing/deleting the sensor.

Also, we added tests functions for the backend and for the frontend for editing the sensor (the one for deleting was already there).

This PR, solves #112, #177, #176

codecov-commenter commented 7 months ago

Codecov Report

Attention: 21 lines in your changes are missing coverage. Please review.

Comparison is base (305a1fe) 89.19% compared to head (927ec4f) 88.95%.

:exclamation: Current head 927ec4f differs from pull request most recent head e890c4f. Consider uploading reports for the commit e890c4f to get more accurate results

Files Patch % Lines
dtbase/webapp/app/sensors/routes.py 43.33% 17 Missing :warning:
dtbase/backend/api/sensor/routes.py 80.00% 3 Missing :warning:
dtbase/core/sensors.py 87.50% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #183 +/- ## ========================================== - Coverage 89.19% 88.95% -0.25% ========================================== Files 75 75 Lines 5036 5125 +89 ========================================== + Hits 4492 4559 +67 - Misses 544 566 +22 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.