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(manager-react-component): add service state badge component #15065

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

tibs245
Copy link
Contributor

@tibs245 tibs245 commented Jan 23, 2025

Question Answer
Branch? develop
Bug fix? no
New feature? yes
Breaking change? no
Tickets Fix #MANAGER-16641 (But finaly unused for the moment)
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 feat(common-translations): add services label state #15061
  • 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

#15061

@tibs245 tibs245 requested review from a team as code owners January 23, 2025 08:18
@github-actions github-actions bot added the feature New feature label Jan 23, 2025
@tibs245 tibs245 force-pushed the feat/mrc-add-service-state-badge branch from 76c8b98 to db435f1 Compare January 23, 2025 13:27
darsene
darsene previously approved these changes Jan 23, 2025
stif59100
stif59100 previously approved these changes Jan 23, 2025
@@ -0,0 +1,31 @@
import fr_FR from '@ovh-ux/manager-common-translations/dist/@ovh-ux/manager-common-translations/service/Messages_fr_FR.json';
Copy link
Contributor

Choose a reason for hiding this comment

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

this seems risky and also the tests fails because of it, is it the way we should import it ? I don't think so but I'm not sure

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think we need build the module before test.

lionel95200x
lionel95200x previously approved these changes Jan 23, 2025
@tibs245 tibs245 dismissed stale reviews from lionel95200x, stif59100, and darsene via d8dc2f7 January 23, 2025 13:59
@tibs245 tibs245 force-pushed the feat/mrc-add-service-state-badge branch from db435f1 to d8dc2f7 Compare January 23, 2025 13:59
@github-actions github-actions bot added the continuous-integration Changes to our CI configuration files and scripts label Jan 23, 2025
@tibs245 tibs245 linked an issue Jan 23, 2025 that may be closed by this pull request
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
continuous-integration Changes to our CI configuration files and scripts feature New feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[manager-components]: Add service state badge
5 participants