Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add integration article to Azure API management #1503

Merged
merged 2 commits into from
May 24, 2021

Conversation

CodeMonkeyLeet
Copy link
Contributor

Thank you for helping make the Dapr documentation better!

Please follow this checklist before submitting:

  • Read the contribution guide
  • Commands include options for Linux, MacOS, and Windows within codetabs
  • New file and folder names are globally unique
  • Page references use shortcodes instead of markdown or URL links
  • Images use HTML style and have alternative text
  • Places where multiple code/command options are given have codetabs

In addition, please fill out the following to help reviewers understand this pull request:

Description

Add a link article under Developing Applications/Integrations for Azure API Management.

Issue reference

Fixes #1303

@CodeMonkeyLeet CodeMonkeyLeet requested a review from a team as a code owner May 21, 2021 18:46
@AaronCrawfis AaronCrawfis linked an issue May 21, 2021 that may be closed by this pull request
title: "Dapr integration policies for Azure API Management"
linkTitle: "Azure API Management"
description: "Publish APIs for Dapr services and components through Azure API Management policies"
weight: 3000
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
weight: 3000
weight: 5000

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@AaronCrawfis Should this be 6000 to put it below the existing "Cloud Providers" link instead? (Or should the Could Providers _index.md be bumped to 6000 if you want an explicitly ordered list?)

Copy link
Collaborator

Choose a reason for hiding this comment

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

Great point. We should actually place all the Azure ones under cloud providers folder. I'll merge this as is and PR a separate change for restructuring the folder

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Article needed to link to Azure APIM
2 participants