Skip to content

Conversation

@HeleneAmouzou
Copy link
Contributor

@HeleneAmouzou HeleneAmouzou commented Nov 14, 2025

What does this PR do?

  • Adds Jenkins dashboard to index.html,
  • Declines cc-addon-header and cc-addon-info for Jenkins product.

How to review?

  • Check the Jenkins' commits ,
  • Check the code,
  • Play with demo-smart.

The .md documentation files has been coded by IA, even though I have checked and adjusted afterwards, please pay attention to it.

TO DO

  • Rebase on master once the Elastic PR is merge to remove Elastic commits.

@HeleneAmouzou HeleneAmouzou self-assigned this Nov 14, 2025
@HeleneAmouzou HeleneAmouzou force-pushed the dashboard-addons/jenkins branch from 8a7b4b1 to 3aefac4 Compare November 14, 2025 16:44
@github-actions
Copy link
Contributor

🔎 A preview has been automatically published : https://clever-components-preview.cellar-c2.services.clever-cloud.com/dashboard-addons/jenkins/index.html.

This preview will be deleted once this PR is closed.

Copy link
Member

@Galimede Galimede left a comment

Choose a reason for hiding this comment

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

Hey Hélène, thanks for the smart! 💪

Overall LGTM, there's just mostly nitpicks below, GG! 👏

updateComponent('state', {
type: 'loaded',
version: {
stateType: 'up-to-date',
Copy link
Member

Choose a reason for hiding this comment

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

question: I'm not sure to understand this up-to-date thing, are we always setting it to up-to-date because APIs currently don't provide this information?

Copy link
Contributor Author

@HeleneAmouzou HeleneAmouzou Nov 18, 2025

Choose a reason for hiding this comment

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

Some products can be updated by users, others can not, we set up-to-date for those that can not be updated by users

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.

3 participants