-
Notifications
You must be signed in to change notification settings - Fork 22
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(cc-logs-addon-runtime): init component #1359
Conversation
🔎 A preview has been automatically published : https://clever-components-preview.cellar-c2.services.clever-cloud.com/addon-logs/index.html. This preview will be deleted once this PR is closed. |
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.
Great job @pdesoyres-cc I've played a little bit with the component and checked the code, couldn't find much to say, only 2 nitpicks 👍
I think it shows the logs UI framework you've built is reaching maturity, well done 🙌
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.
LGTM! Great highlight of the log groundwork you built.
src/components/cc-logs-addon-runtime/cc-logs-addon-runtime.smart.js
Outdated
Show resolved
Hide resolved
47c0bc0
to
822cd9f
Compare
🔎 The preview has been automatically deleted. |
What this PR do?
This PR adds
cc-logs-addon-runtime
component that displays addon logsHow to review