Azure / LogicAppsUX

https://learn.microsoft.com/azure/logic-apps
MIT License
76 stars 81 forks source link

Dynamic Lists in a Dynamic UI Element is not performant #3708

Closed siddharth-ms closed 10 months ago

siddharth-ms commented 11 months ago

Describe the Bug with repro steps

We are trying to generate a UI like:

Screenshot 2023-11-16 184656

We have an element rfcName with Dynamic List, Ref: Manifest below: Screenshot 2023-11-16 185901 Where the contents of the elements are fetched by a dynamicInvoke call like:

https://management.azure.com/subscriptions/f34b22a3-2202-4fb1-b040-1332bd928c84/resourceGroups/siddharth-imp/providers/Microsoft.Web/sites/sid-sap-testbox/hostruntime/runtime/webhooks/workflow/api/management/operationGroups/sap/operations/listRfcs/dynamicInvoke?api-version=2018-11-01

The element works just fine when it's a static element, but performance is much slower and the page becomes un-responsive when the element is created through a dynamic call, Here the field rfcName is generate as a result of Dynamic selection of bodyType.

Selecting bodyType as Dynamic makes below call:

curl 'https://management.azure.com/subscriptions/f34b22a3-2202-4fb1-b040-1332bd928c84/resourceGroups/siddharth-imp/providers/Microsoft.Web/sites/sid-sap-testbox/hostruntime/runtime/webhooks/workflow/api/management/operationGroups/sap/operations/getCallRfcInputVarientSchemaSwagger/dynamicInvoke?api-version=2018-11-01' \
  -H 'x-ms-user-agent: LogicAppsDesigner/2.75.0 (host azure-logic-apps-client ; source DesignerStandard)' \
  --data-raw $'{"parameters":{"bodyType":"Dynamic"},...' \
  --compressed

and Response: Screenshot 2023-11-16 190321 Which results in creation of dynamic field rfcName. When is element is top level element, and not a dynamic element, it is able to handle list of RFCs but not in case of Dyanmic element.

What type of Logic App Is this happening in?

Standard (Portal)

Are you using new designer or old designer

New Designer

Did you refer to the TSG before filing this issue? https://aka.ms/lauxtsg

Yes

Workflow JSON

No response

Screenshots or Videos

No response

Browser

Edge

Additional context

No response

AB#25863872

preetriti1 commented 11 months ago

Just to be clear, you are not seeing dynamic list being populated for rfcName when that is a dynamic property returned by body type dynamic call?

siddharth-ms commented 11 months ago

No, we see a REST call to dynamicInvoke fetching all the RFCs. But while populating the UI stops responding, and the Edge browser suggests to kill the page.