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

feat(pci-common): add pci saving plan banner #14739

Open
wants to merge 10 commits into
base: maintenance/manager-pci-common-v0.x
Choose a base branch
from

Conversation

ppprevost
Copy link
Contributor

Question Answer
Branch?
Bug fix? no
New feature? yes
Breaking change? no
Tickets Fix #...
License BSD 3-Clause
  • Try to keep pull requests small so they can be easily reviewed.
  • Commits are signed-off
  • Only FR translations have been updated
  • Branch is up-to-date with target branch
  • Lint has passed locally
  • Standalone app was ran and tested locally
  • Ticket reference is mentioned in linked commits (internal only)
  • Breaking change is mentioned in relevant commits

Description

Related

lionel95200x
lionel95200x previously approved these changes Dec 30, 2024
@ppprevost ppprevost changed the base branch from maintenance/manager-pci-common-v0.x to ci/filter-tests-and-build December 31, 2024 09:10
Pierre-Philippe added 9 commits December 31, 2024 10:24
Signed-off-by: Pierre-Philippe <[email protected]>
Signed-off-by: Pierre-Philippe <[email protected]>
Signed-off-by: Pierre-Philippe <[email protected]>
Signed-off-by: Pierre-Philippe <[email protected]>
Signed-off-by: Pierre-Philippe <[email protected]>
@ppprevost ppprevost force-pushed the feat/add-pci-saving-plan branch from 8f1f810 to 24fa9dd Compare December 31, 2024 09:25
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
10.0% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@ppprevost ppprevost changed the base branch from ci/filter-tests-and-build to maintenance/manager-pci-common-v0.x December 31, 2024 09:29
@ppprevost ppprevost dismissed lionel95200x’s stale review December 31, 2024 09:29

The base branch was changed.

@ppprevost ppprevost requested review from a team as code owners December 31, 2024 09:29
@fredericvilcot fredericvilcot self-requested a review December 31, 2024 09:34
Copy link
Contributor

@fredericvilcot fredericvilcot left a comment

Choose a reason for hiding this comment

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

👍🏻

const navigateToSavingsPlan = async () => {
await navigateTo(
'public-cloud',
`#/pci/projects/${projectURL}/savings-plan`,
Copy link
Contributor

Choose a reason for hiding this comment

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

shouldn't this come from shell context ?

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