-
Notifications
You must be signed in to change notification settings - Fork 23
feat(cc-addon-*.smart-jenkins): init #1611
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
base: master
Are you sure you want to change the base?
Conversation
8a7b4b1 to
3aefac4
Compare
|
🔎 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. |
Galimede
left a comment
There was a problem hiding this 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! 👏
src/components/cc-addon-header/cc-addon-header.smart-jenkins.md
Outdated
Show resolved
Hide resolved
src/components/cc-addon-header/cc-addon-header.smart-jenkins.js
Outdated
Show resolved
Hide resolved
src/components/cc-addon-header/cc-addon-header.smart-jenkins.js
Outdated
Show resolved
Hide resolved
| updateComponent('state', { | ||
| type: 'loaded', | ||
| version: { | ||
| stateType: 'up-to-date', |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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
What does this PR do?
index.html,cc-addon-headerandcc-addon-infofor Jenkins product.How to review?
demo-smart.The
.mddocumentation files has been coded by IA, even though I have checked and adjusted afterwards, please pay attention to it.TO DO
ElasticPR is merge to removeElasticcommits.