OED icon indicating copy to clipboard operation
OED copied to clipboard

CSV meter upload validates values

Open huss opened this issue 1 year ago • 16 comments

Is your feature request related to a problem? Please describe.

The create meter page stops admins from entering bad values by constraining choices and/or checking before saving. It would be good if the CSV meter upload page did similar checks.

Describe the solution you'd like

See above.

Describe alternatives you've considered

No checking

Additional context

None

huss avatar Apr 09 '24 21:04 huss

I will work on this issue.

mikedivine avatar Aug 13 '24 05:08 mikedivine

I can work on this issue.

ethanernst11 avatar Oct 18 '24 20:10 ethanernst11

I can work on this issue.

I think this is okay since @mikedivine is not working on this at this time. If you need any help on this then let me know.

huss avatar Oct 18 '24 20:10 huss

I have a quick question. Where can I find the code for the create meter page that stops admins from entering bad values?

ethanernst11 avatar Oct 20 '24 18:10 ethanernst11

I have a quick question. Where can I find the code for the create meter page that stops admins from entering bad values?

See src/client/app/components/meters/CreateMeterModalComponent.tsx (there is an edit equivalent). There are a number of checks here that are done in different ways (different ways is not needed for this issue). As examples, it validates GPS, can't have cumulative reset if not cumulative, set identifier if name given, no blank name, .... Let me know if you need anything else.

huss avatar Oct 20 '24 19:10 huss

Is the csv meter upload also creating a meter?

ethanernst11 avatar Oct 22 '24 15:10 ethanernst11

Is the csv meter upload also creating a meter?

In general, the CSV meter upload creates meter(s). It is possible to edit a single meter but that is less common. If you have not seen it yet, you may find the documentation at https://openenergydashboard.org/helpV1_0_0/adminCsvImport/ useful. It is for v1.0.0 so is a little out of date with development but still useful.

huss avatar Oct 22 '24 15:10 huss

I apologize but this ended up being more complicated than expected and I do not have enough time currently to work on it. So I will not be able to continue working on this issue for finish it. I am sorry for any inconvenience.

ethanernst11 avatar Oct 24 '24 14:10 ethanernst11

I apologize but this ended up being more complicated than expected and I do not have enough time currently to work on it. So I will not be able to continue working on this issue for finish it. I am sorry for any inconvenience.

It's fine and thanks for letting us know. If you want to work on something in the future then please just let us know (and we can help find a task that suits your situation if that helps).

huss avatar Oct 24 '24 14:10 huss