Rothamsted-Ecoinformatics / farm_rothamsted

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

FarmOS 3.0: Moving plant assets #619

Closed aislinnpearson closed 7 months ago

aislinnpearson commented 7 months ago

@mstenta I think I found a bug where you can't move a plant asset from the edit page (the move button from the asset page does work, see this log and its revision history: https://pr-12-qcotibq-2mu45anerqtvq.uk-1.platformsh.site/asset/239).

At first I thought it was because I hadn't initially specified a location when creating the plant asset, but I had the same problem when the asset had a location.

Images of the error messages below.

image

image

image

image

aislinnpearson commented 7 months ago

Spoke to @mstenta about this and is Rothamsted specific.

paul121 commented 7 months ago

This feature needed to be updated to use the new movement quick form actions. A simple fix, also made a change so that this appears in the "Location" tab for the asset.