CleverCloud / clever-components

Collection of Web Components by Clever Cloud
https://www.clever-cloud.com/doc/clever-components/
Apache License 2.0
215 stars 19 forks source link

feat(cc-logs-application-view): init component #980

Closed pdesoyres-cc closed 3 months ago

pdesoyres-cc commented 4 months ago

Fixes #967 Fixes #965 Fixes #964 Fixes #1019 Fixes #983

github-actions[bot] commented 4 months ago

🔎 A preview has been automatically published : https://clever-components-preview.cellar-c2.services.clever-cloud.com/cc-logs-application-view/init/index.html.

This preview will be deleted once this PR is closed.

florian-sanders-cc commented 3 months ago

Forgot to say:

Logs instances

Back again about the instance indexes thing :joy: I've wrapped my head around the fact we show indexes and their use but in the long term, I think we could do better for our customers: the APIs and the UIs could rely on a concept of "primary" instance (which would correspond to the instance #0). This way we could add a badge like "primary" or "cron" or whatever next to the instance name. I think it would be a better solution to the "how do we tell people which instance may contain the cron logs?" because it would only show the information we need instead of showing information not relevant for users (indexes of the other instances).

Anyway, as I said, I've wrapped my mind around it, maybe people won't really be confused about this like I was but I just wanted to suggest another idea in the long term.

florian-sanders-cc commented 3 months ago

I just realized I had not tested form errors within the "date range" section:

github-actions[bot] commented 3 months ago

🔎 The preview has been automatically deleted.