[Create/edit survey] Allow data collectors to optionally choose between draw point or polygon tasks

[gino-m]

When doing EUDR reporting, for smaller plots farmers are only required to report a point. Organizers should have the option of punting the decision of geometry type to the data collector. To make this work, we could “Drop a pin” and “Draw or walk a perimeter” into a single task, with two names depending on context:

• “Map a new site” for “Add LOI tasks”
• “Annotate site on map” for other “advanced” geometry tasks

In both cases, the survey organizer must specify one or more “Allow data collectors to:” options:

• “Drop a pin”
• “Draw or walk perimeter”

If the survey organizer chose >1 geometry of these options, collectors will be prompted to choose which they want to do (e.g., in modal with pin and polygon icons).

@vittorino @amegantz for designs.

Blocked by google/ground-android#2469

[amegantz]

Obsolete design redacted.

[gino-m]

@vittorino I think we'd either want the dropdown or the chips (possible with checkboxes to indicate state. Can you update with the final version?

@amegantz For consistency with other field labels (eg 'Instructions") it should probably be in heading voice, eg "Allowed mapping actions" or simiar.

[amegantz]

I think "Mapping actions" works for the heading. Updated the Figma

[vittorino]

Here is the updated mock:

Obsolete design redacted.

[gino-m]

@amegantz I think adding "Allowed" makes it clearer that the user has a choice, not that they need to perform all the actions specified there. If it becomes too long, we can drop "Mapping" since it should be clear from context.

[amegantz]

[gino-m]

@lecrabe Punting this to GA release since there's a workaround for smaller plots (draw a small polygon)

[gino-m]

As per conversation today with @jo-spek, we should also allow users to select "Capture location" as a geometry task to use when adding new sites. Please cmiiw.

[gino-m]

@rfontanarosa Please coordinate with @anandwana001 on google/ground-android#2469.

The DrawGeometry proto already has allowed_methods which can be used for this purpose. For this FR I would leave out @jo-spek's request to allow adding a LOI using the device location for now. Users can work around this by adding a Capture Location task manually to the job.

I suggest @anandwana001 does google/ground-android#2469 before implementing this on web.

[gino-m]

@rfontanarosa and I spoke. Suggestions:

• Let's punt on "Annotate site" for now, focus on "Map a new site"
• The new site task will be added in the last position like when adding other tasks
• Update the "free form data collection" label for consistency
• We can delete the old commented out Add Drop Pin and Walk perimeter code
• Let's also avoid adding "Capture location" for adding a new site for now (see previous comment)

@vittorino @amegantz What text should appear in the dropdown when multiple actions are selected?

[vittorino]

We have 2 options: 1) display all values separated by a comma and truncated or 2) display just multiple
I'm a little more inclined towards the second option, but fine with either one

[gino-m]

We have 2 options: 1) display all values separated by a comma and truncated or 2) display just multiple I'm a little more inclined towards the second option, but fine with either one

Let's go with Multiple, since it scaled better to other languages and avoids truncation.

[jcqli]

Adding product label to discuss behavior between "Tasks" and "Sites". Toggling free form data collection in "Sites" also adds a "Map a new site" task, is this correct?

[gino-m]

@rfontanarosa Can we close this?

[gino-m]

As per conversation today with @jo-spek, we should also allow users to select "Capture location" as a geometry task to use when adding new sites. Please cmiiw.

+100 from @n-clinton

[gino-m]

@rfontanarosa Needs to hide/show "Map a site" button depending on whether task is already present.