cachethq / core

🚦 The core of Cachet, the open-source status page system.
https://cachethq.io
Other
159 stars 36 forks source link

Single Incident View Improvements #70

Open jbrooksuk opened 1 month ago

jbrooksuk commented 1 month ago

Couple of improvements to the incident view page.

RahatAhmed171 commented 1 month ago

Hi, how do i get startee contributing to this issue. Wanna give it a try

jbrooksuk commented 1 month ago

@RahatAhmed171 clone down the cachethq/core repository. Run composer install and npm install. Now run npm run dev and in a new terminal run composer start. That should do it!

RahatAhmed171 commented 1 month ago

error Dashboard doesnt show up. It returns 404 not found

jbrooksuk commented 1 month ago

https://x.com/jbrooksuk/status/1845087763835633898?s=46

Hope this helps!

RahatAhmed171 commented 1 month ago

The dashboard link at the top doesn’t work at all. I inspected the href, the url looks like this:

<a href="/\dashboard" class="text-sm font-medium text-zinc-800 transition hover:text-zinc-700 dark:text-zinc-400 dark:hover:text-zinc-300 sm:text-base">Dashboard</a>
jbrooksuk commented 1 month ago

How are you running Cachet? What's CACHET_PATH set to? I can't recreate this locally or at v3.cachethq.io

RahatAhmed171 commented 1 month ago

so there is a dashboard function in src/cachet.php. I made some changes and and its working now (weird). Regarding the single incident page, correct me if i am wrong, you want to display the component that is attached to a incident? single_incident How do you want to display it? below the incident updates?

jbrooksuk commented 1 month ago

I'm glad you're sorted!

Well, we currently show an unstyled list of components when an incident is attached to one or more. I think we should firstly improve that UI (in the timeline) and then figure out a way that can be improved in this view.

RahatAhmed171 commented 1 month ago

That makes sense. Do you have any specific ideas on what changes would make the component list look more user-friendly?

jbrooksuk commented 1 month ago

The design by Jump24 actually had it quite similar to what we have: CleanShot 2024-10-16 at 13 15 25@2x

RahatAhmed171 commented 1 month ago

This is only one component above the incident title in your design. How you're gonna display If there's more than one component it shows as comma-separated list? cim

jbrooksuk commented 1 month ago

Yeah, I think so.

Also, we could take this a step further... what if instead of making them a string, they were a block of individual Blade components that had the status icon and a tooltip or something for interactivity?