Azure / logicapps

Azure Logic Apps labs, samples, and tools
MIT License
366 stars 302 forks source link

`XML transform` action - missing `transform options` input in Standard #926

Closed prijeshramani closed 7 months ago

prijeshramani commented 1 year ago

Describe the Bug with repro steps

While using the Standard Logic app, I am not able to see the Transform option in the XML transform. I have to apply "transformOptions": "applyXsltOutputAttributes" but I am not finding that option on the shape.

I had to add that line explicitly in the code view. Can you please fix this in new designer of Standard logic App.

What type of Logic App Is this happening in?

Standard (Portal)

Are you using Preview Designer or GA Designer

Generally Available

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

MS Edge

Additional context

No response

AB#24383621

AB#26009411

rllyy97 commented 1 year ago

Hi @prijeshramani, thanks for the bug report. Just to clarify, are you asking for the transformOptions property to also appear in Standard logic apps, or reporting that the Consumption version is not working?
I am working on the issue with this property in Consumption but I wanted to verify since you marked the issue as Standard

prijeshramani commented 1 year ago

@rllyy97 - yes i would like to have the dropdown in the standard logic app as well. currently there is no dropdown. Only workaround is to add the line via code view so can you please make the change in standard logic app

prijeshramani commented 1 year ago

Also please change the label to standard

prijeshramani commented 1 year ago

It is not fixed, please reopen

Eric-B-Wu commented 1 year ago

When we close an issue, in general it takes 2 weeks or so to release into all regions as we must follow a deployment schedule. If this is still appearing in the next release which is set to be completed end of next week, please leave another comment and we will revisit this issue

prijeshramani commented 1 year ago

Hi @eric wu, I got your point on the process, But my point is that issue is merged with the consumption, and thus i have commented on it Actual issue is related to standard logic app So please let me know if this is considered for it and solved accordingly

On Tue, 4 Jul 2023 at 10:27 PM, Eric Wu @.***> wrote:

When we close an issue, in general it takes 2 weeks or so to release into all regions as we must follow a deployment schedule. If this is still appearing in the next release which is set to be completed end of next week, please leave another comment and we will revisit this issue

— Reply to this email directly, view it on GitHub https://github.com/Azure/logicapps/issues/926, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADXDJFZGAUEMB64BX5IEU33XORDRDANCNFSM6AAAAAAZVEJLQY . You are receiving this because you were mentioned.Message ID: @.***>

-- Regards

PRIJESH H. RAMANI

Eric-B-Wu commented 1 year ago

@prijeshramani the new designer is meant to be have a unified experience across all surfaces and SKUs, including that between standard and consumption, so in this case, we would be fixing for both consumption and standard

rllyy97 commented 1 year ago

@prijeshramani Just to clear up what Eric was saying, this action is different between standard and consumption. Not considering new designer, legacy consumption does show the transform options property and legacy standard does not show it. Before we add that parameter to the standard version of the operation I need to check with another team to confirm that it's supported. For now I'm going to reopen, but this is a feature request and it will be low priority for us.

rllyy97 commented 1 year ago

I've discussed with the other team and due to bandwidth constraints we are not going to be working on this at the moment. However, I've been told the behavior is supported on the backend currently, so if you want to add it manually through code it should theoretically work.

hartra344 commented 10 months ago

Since this is a manifest issue and not a UX issue, I'm moving this to the our backend tracking repo.

github-actions[bot] commented 9 months ago

This issue is stale because it has been open for 45 days with no activity.

prijeshramani commented 9 months ago

Hi Team, Any update

rllyy97 commented 9 months ago

Hi @prijeshramani, I just checked the Transform XML action for both the Transform options parameter and option you specified, and I am seeing both on this action. Are you still having issues with the action?

image

github-actions[bot] commented 7 months ago

This issue is stale because it has been open for 45 days with no activity.

github-actions[bot] commented 7 months ago

This issue was closed because it has been inactive for 14 days since being marked as stale.