Gold872 / elastic-dashboard

A simple and modern dashboard for FRC
MIT License
71 stars 13 forks source link

Added option to duplicate field json data for more ease of creating additional fields #64

Closed DanPeled closed 2 weeks ago

DanPeled commented 1 month ago

Will like if the reviewer will be able to add such that it will not require an app restart, couldn't manage to do it by myself

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 0% with 58 lines in your changes missing coverage. Please review.

Project coverage is 45.63%. Comparing base (8bc547d) to head (d3a3640).

Files Patch % Lines
...b/widgets/nt_widgets/multi-topic/field_widget.dart 0.00% 58 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #64 +/- ## ========================================== - Coverage 45.85% 45.63% -0.22% ========================================== Files 74 74 Lines 7541 7577 +36 ========================================== Hits 3458 3458 - Misses 4083 4119 +36 ```

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

Gold872 commented 1 month ago

I don't see the need for this feature since right now adding a new field requires modifying the assets folder manually, and it's much easier to copy the necessary files in file explorer than have a button in the app to do it, especially since it's much harder to test if it's done by the app