search

USACE / instrumentation

Instrumentation project issue tracking and project planning
MIT License
5 stars 1 forks source link

Add Instrument Error #4

Closed lucasb16-exe closed 3 years ago

lucasb16-exe commented 3 years ago

Throws exception and displays white screen on attempting to add instrument.

Associated code according to debug console is highlighted. Find screenshot of error message in debug console attached.

errorCode

Input for add instrument fields throwing error - Project: Upper Appleton Locks and Dam Name: UD-B-1 Type: Piezometer Status: Abandoned Status current as of: 12/14/2020 Station: * Offset: Use Projection: WGS 84 (DD) Longitude: -88.411711 Latitude: 44.253386 adderror

Error just started occurring today, any input would be appreciated, thanks!

KevinJJackson commented 3 years ago

@lucasb16-exe - Hey Lucas, when was the last time you tried to add an instrument prior to this? I can take a look at some of the recent UI changes to see if something recently added caused this to go down or if you hit a weird snag that needs cleaned up.

lucasb16-exe commented 3 years ago

@KevinJJackson

Hi Kevin, I had successfully added 4 instruments on 12/7/2020 to the Chicago Lock project. If there's anything you'd like me to try for troubleshooting, let me know. I'm looking at some of the changes from the upload-cleanup enhancement merge to see if there'd be some issue caused there.

KevinJJackson commented 3 years ago

At first glance, I'd assume this line: https://github.com/USACE/instrumentation-ui/blob/develop/src/app-components/domain-select.js#L20

Think that just needs to be set to: onChange={onChange} and it'd probably work as expected.

Let me see if I can test this locally first

KevinJJackson commented 3 years ago

Yep, verified that was the culprit. I'll merge a hotfix for that line of code so that gets deployed ASAP

KevinJJackson commented 3 years ago

@lucasb16-exe This should be deployed on dev in a few minutes or so. if you could try to reproduce again once it's deployed and report your findings here that'd be great!

lucasb16-exe commented 3 years ago

Instruments successfully adding. Closing issue, thanks @KevinJJackson !