search

microsoft / azure-api-management-monetization

MIT License
54 stars 31 forks source link

General improvements #20

Closed Smartitect closed 2 years ago

Smartitect commented 2 years ago

Changes made during creation of walk through videos for Stripe and Adyen deployments:

  1. "Deploy to Azure" button URL was not correct - changed in both Adyen and Stripe documents;
  2. Container URL - changed in main.bicep to point to public container repository rather than to private GitHub repository;
  3. Race condition in deployment - resolved by adding additional dependency in main.bicep;
  4. Transpiled a new JSON deployment based on changes in two steps above;
  5. Updated documentation to clarify deployment process and to accommodate recent changes to navigation in provider portals.

These changes will also address #19.

mikebudzynski commented 2 years ago

@Smartitect, do we need to publish a new container image?

MikeEvansLarah commented 2 years ago

@Smartitect, do we need to publish a new container image?

@mikebudzynski no need for a new image, there haven't been any changes to the web application