Skip to content

Allow publishing to draft plans in Azure#4

Merged
JAVGan merged 1 commit into
mainfrom
draft_plans
Jul 10, 2024
Merged

Allow publishing to draft plans in Azure#4
JAVGan merged 1 commit into
mainfrom
draft_plans

Conversation

@JAVGan

@JAVGan JAVGan commented Jul 4, 2024

Copy link
Copy Markdown
Collaborator

This commit creates a new method named get_product_plan_by_name which seeks the desired plan name in the product for all available Azure Targets: draft, live, preview.

It also changes the publish method to use this new method to prevent a bug which a product is found but not the desired plan because it is in the draft target.

Refers to SPSTRAT-296

This commit creates a new method named `get_product_plan_by_name` which
seeks the desired plan name in the product for all available Azure
Targets: `draft`, `live`, `preview`.

It also changes the `publish` method to use this new method to prevent a
bug which a product is found but not the desired plan because it is in
the `draft` target.

Refers to SPSTRAT-296
@JAVGan JAVGan marked this pull request as ready for review July 4, 2024 22:11
@JAVGan

JAVGan commented Jul 4, 2024

Copy link
Copy Markdown
Collaborator Author

@lslebodn @ashwini3326 @jajreidy PTAL

I'm marking it as READY since the tests I'm making indicates it works. You can find more details about it in the respective JIRA.

Note that I also took advantage of this PR to remove some legacy stage_preview features we no longer use, so it fixes 2 issues with one shot.

@JAVGan JAVGan requested review from ashwgit, jajreidy and lslebodn July 4, 2024 22:13

@jajreidy jajreidy left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@JAVGan JAVGan merged commit 1fbe9bd into main Jul 10, 2024
@JAVGan JAVGan deleted the draft_plans branch July 10, 2024 12:49
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.

2 participants