search

Rothamsted-Ecoinformatics / farm_rothamsted

Custom farmOS features for Rothamsted Research.
GNU General Public License v2.0
6 stars 1 forks source link

Plan assets hidden from plan view #613

Closed aislinnpearson closed 7 months ago

aislinnpearson commented 7 months ago

When trying to add an experiment boundary to the plans you get the below error. See here for the actual asset I created (I created two actually with two different KMLs and it happened both times):

https://pr-12-qcotibq-2mu45anerqtvq.uk-1.platformsh.site/asset/41 https://pr-12-qcotibq-2mu45anerqtvq.uk-1.platformsh.site/asset/230

image

The error appeared when navigating to the plot geometry tab: https://pr-12-qcotibq-2mu45anerqtvq.uk-1.platformsh.site/plan/3/update-plot-geometry

image

But I had no actual problems with uploading the plot geoms: https://pr-12-qcotibq-2mu45anerqtvq.uk-1.platformsh.site/plan/3/plots

The error is also on the page with the experiment variables: https://pr-12-qcotibq-2mu45anerqtvq.uk-1.platformsh.site/plan/3/variables

image

But again the actual variables uploaded just fine: https://pr-12-qcotibq-2mu45anerqtvq.uk-1.platformsh.site/plan/3/plots

Finally, the experiment boundary gets added as an asset but does not display: image

When trying to replace the KML file on the experiment boundary asset itself I get this error when trying to add a new KML file:

image

paul121 commented 7 months ago

The error appeared when navigating to the plot geometry tab

@aislinnpearson thanks, you found another small bug in one of our dependencies! I believe this is just a warning and not a major issue, but good to know. In the demo environment I have warning messages turned on because it helps to catch small bugs like this but normally these are turned off in production.

Finally, the experiment boundary gets added as an asset but does not display:

Yes... it seems like none of the Plan assets are being displayed. I think I know why.

paul121 commented 7 months ago

Yes, I found the fix. The experiment boundary was being created just fine, you just couldn't see it being referenced on the plan. Fixed, although I may not get the demo updated right away.

aislinnpearson commented 7 months ago

Fixed, although I may not get the demo updated right away.

No worries @paul121 - if you've fixed and it isn't appearing your end then it I don't think we need to update the demo site. :)