Closed ryangroth5 closed 3 years ago
I updated this to account for the new verify/unverify PR that was merged.
@ngottlieb saw your comment, no, don't mind at all. I've been using this to test the server-side changes though, so having an "updated" version of reach create without geometry is still helpful. delete this PR if you have one that is better.
@ngottlieb this also tweaks error detail reporting on reach create failure which might be worth salvaging in some way
Note: this was supposed to work towards https://github.com/AmericanWhitewater/wh2o/issues/2228.
It's been replaced by https://github.com/AmericanWhitewater/wh2o-vue/pull/351
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...) This will suppress data going to reachCreate that does not have a valid geometry or length enabling reaches to be created without geometries.
What is the current behavior? (You can also link to an open issue here) invalid data goes to reach create
What is the new behavior (if this is a feature change)? invalid data is filtered from going to reach create
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?) no
Other information: logged a bug about not being able to remove geometries after they are created as low priority.