Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature forms fix #317

Open
wants to merge 1 commit into
base: v.next
Choose a base branch
from
Open

Feature forms fix #317

wants to merge 1 commit into from

Conversation

shubham7109
Copy link
Collaborator

@shubham7109 shubham7109 commented Feb 5, 2025

Description

PR to patch feature forms sample with sample data changes

Issue

@markjdugger
Copy link
Collaborator

markjdugger commented Feb 5, 2025

@shubham7109
This fix does not address the defective feature form definition.
If you run the app in this PR and tap on a place, you will see a Feature form editor that has dozens of fields. However, the feature form definition in the web map has only 12 fields:

image

So...I think you're getting the default feature form, which is not what the sample, I suspect, aims to show. The API Ref for FeatureForm says:

If the feature's com.arcgismaps.mapping.layers.FeatureLayer, com.arcgismaps.data.ArcGISFeatureTable, or the com.arcgismaps.mapping.layers.SubtypeSublayer specific to the feature's com.arcgismaps.data.FeatureSubtype has been authored with a FeatureFormDefinition, then this authored FeatureFormDefinition will be used to create the FeatureForm. If a FeatureFormDefinition is not found, a default definition is generated and used to create the FeatureForm.

Copy link
Collaborator

@markjdugger markjdugger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This fix provides a default feature form definition, instead of the feature form definition specified in the web map's JSON. See my comment above for details.

It's up to you whether you think this is appropriate for the sample.

@markjdugger
Copy link
Collaborator

@shubham7109
I checked with Vivian about Swift version of this sample. She has the same results as you: a default feature form that displays all the fields in the table/layer. That's over 90!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants