Setup id at runtime? about fastapi-amis-admin HOT 6 CLOSED

I can not determine the specific reason, you can set a program breakpoint here to debug, see whether the code is running normally.

from fastapi-amis-admin.

i tried add some event logic, and this requires to set 'id' property for fields because that used to identify field. If not setup id when copypaste to editor online (to educate logic and properties) editor autogenerate id. So I little confuse id require or not?

    async def get_form_item(
        self, request: Request, modelfield: ModelField, action: CrudEnum
    ) -> Union[FormItem, SchemaNode, None]:
        item = await super().get_form_item(request, modelfield, action)

        if item.name == Trigger.event.key:  # noqa
            item.id = item.name

        if item.name == Trigger.target.key:  # noqa
            item.id = item.name  # noqa
            item.selectFirst = True  # noqa
            item.clearValueOnHidden = True  # noqa
            item.visibleOn = '${event!="STANDALONE"}'

            item.source = {
                "url": "TriggerAdmin/target_items/list?tet=${event}",
                "sendOn": 'event'
            }

        return item

maybe just autosetup id field at runtime? i newbe, maybe I'm not right, what you think about this ?

from fastapi-amis-admin.

Are you suggesting that the fastapi-amis-admin source code set an id for form items by default?

from fastapi-amis-admin.

because when insert to editor id autocreate, example:

"id": "u:57b6fb4bc910",

from fastapi-amis-admin.

The default id is not generated for two main reasons:

1. I want the json data to be as concise as possible while the core functions are working properly.
2. If id is set to name, it is not certain that conflicts will occur if the amis front-end has multiple form components.

from fastapi-amis-admin.

you're right, I think the demo requires a good extending
I will help with this when being possible, I like you library, but so many dark places)))

from fastapi-amis-admin.