search

MicrosoftDocs / azure-docs

Open source documentation of Microsoft Azure
https://docs.microsoft.com/azure
Creative Commons Attribution 4.0 International
10k stars 21k forks source link

Update api-management-api-import-restrictions.md #122244

Open anjalidivitha opened 1 week ago

anjalidivitha commented 1 week ago

By default, APIM tries to convert all required query parameters into Url template parameters, and Url template parameters must be case insensitive unique.

prmerger-automator[bot] commented 1 week ago

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

learn-build-service-prod[bot] commented 1 week ago

Learn Build status updates of commit 037866a:

:white_check_mark: Validation status: passed

File Status Preview URL Details
articles/api-management/api-management-api-import-restrictions.md :white_check_mark:Succeeded

For more details, please refer to the build report.

For any questions, please:

Court72 commented 1 week ago

@dlepow

Can you review the proposed changes?

When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.

label:"aq-pr-triaged"

@MicrosoftDocs/public-repo-pr-review-team

dlepow commented 1 week ago

@anjalidivitha - Thanks for the PR. The addition seems to duplicate the previous line in part which mentions "unique". Could you add the "case-insensitive" part to the previous line instead, e.g. "

  • OpenAPI doesn't support this discrimination, so we require parameter names to be unique within the entire URL template. Names are case-insenstive.
    • " Thanks!

    anjalidivitha commented 4 days ago

    @dlepow, thanks for the feedback. I made changes as suggested. Thanks! :)

    learn-build-service-prod[bot] commented 4 days ago

    Learn Build status updates of commit 390c5ab:

    :white_check_mark: Validation status: passed

    File Status Preview URL Details
    articles/api-management/api-management-api-import-restrictions.md :white_check_mark:Succeeded

    For more details, please refer to the build report.

    For any questions, please:

    dlepow commented 3 days ago

    Thank you @anjalidivitha! LGTM.

    sign-off

    Jak-MS commented 3 days ago

    Due to the upcoming Build conference, all merging and publishing is on hold until the morning of 5/21.

    This PR is ready for merge on 5/21.

    label: "event-triaged-to-10AM"

    label: "missed-content-complete"