Skip to content

Conversation

@watana2
Copy link
Contributor

@watana2 watana2 commented Oct 23, 2025

Added examples demonstrating how to wrap JSON array in an object before conversion. Included two methods (Compose object and concat approach) to generate valid XML output.

Added examples demonstrating how to use the xml() function with JSON data, including two options for converting JSON arrays to XML.
Clarified the input requirements for the xml() function, emphasizing that it expects an object or valid XML string and not a raw array.
@prmerger-automator
Copy link
Contributor

@watana2 : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit e1115fd:

✅ Validation status: passed

File Status Preview URL Details
articles/logic-apps/expression-functions-reference.md ✅Succeeded

For more details, please refer to the build report.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 5e30492:

✅ Validation status: passed

File Status Preview URL Details
articles/logic-apps/expression-functions-reference.md ✅Succeeded

For more details, please refer to the build report.

@v-dirichards v-dirichards requested a review from Copilot October 23, 2025 14:40
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds documentation for converting JSON arrays to XML format in Azure Logic Apps. Since the xml() function doesn't accept raw arrays, the update demonstrates two workarounds for wrapping arrays in objects before conversion.

Key changes:

  • Added Example 4 to the xml() function documentation showing array-to-XML conversion techniques
  • Documented two approaches: using a Compose action with JSON object wrapper, and using concat() to build the wrapper string

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@v-dirichards
Copy link
Contributor

@ecfan

Can you review the proposed changes?

Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator bot added the aq-pr-triaged tracking label for the PR review team label Oct 23, 2025
Copy link
Contributor

@ecfan ecfan left a comment

Choose a reason for hiding this comment

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

Thanks, @watana2!

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 3306c90:

✅ Validation status: passed

File Status Preview URL Details
articles/logic-apps/expression-functions-reference.md ✅Succeeded

For more details, please refer to the build report.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 6e39628:

✅ Validation status: passed

File Status Preview URL Details
articles/logic-apps/expression-functions-reference.md ✅Succeeded

For more details, please refer to the build report.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 6c51b25:

✅ Validation status: passed

File Status Preview URL Details
articles/logic-apps/expression-functions-reference.md ✅Succeeded

For more details, please refer to the build report.

@watana2 watana2 requested a review from ecfan October 25, 2025 02:07
Copy link
Contributor

@ecfan ecfan left a comment

Choose a reason for hiding this comment

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

Thank you!

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 66e9154:

✅ Validation status: passed

File Status Preview URL Details
articles/logic-apps/expression-functions-reference.md ✅Succeeded

For more details, please refer to the build report.

@ttorble
Copy link
Contributor

ttorble commented Oct 27, 2025

@ecfan I'll merge this PR based on your approval. If this article requires further changes, please open a separate pull request.

@ttorble ttorble merged commit 251e1ff into MicrosoftDocs:main Oct 27, 2025
2 checks passed
@watana2 watana2 deleted the patch-1 branch October 27, 2025 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants