Joscha Henningsen

Results 75 comments of Joscha Henningsen

Currently when approving a message it is not simply send to the clients. Instead the clients get notified to reload **all** messages which is essentially a DoS when there are...

I really like the logo you provided! ![image](https://user-images.githubusercontent.com/44805696/194727234-5f973cc2-b56e-4f49-8cd5-a165a2e208ec.png) Considering that it will be quite small when displayed in the header, the edges and padding of the `live` text might be...

I noticed that parts of the svg are just a base64 endoded png, this might be something we could also tackle.

Also, the red of the logo and live indicator should match, I don't think it makes a huge difference which to go forward with.

@0ql looks good as well! Though I probably wouldn't go for a gradient

The little text next to a Livestream on the start page has the background `red-800` https://tailwindcss.com/docs/background-color ![image](https://user-images.githubusercontent.com/44805696/194819354-a6e03380-73f3-4b82-bf41-1ab8ea252278.png)

No need, the logo can be configured without any changes to the code. We'll do a trial run with these. Love them ☺️ On Thu, 3 Nov 2022, 20:21 Kaffee☕,...

This would be where I'd try to fix this with runtime in mind: https://github.com/joschahenningsen/TUM-Live/blob/0f4e2160ea76523450db81a0798ac373dadfcc4c/dao/streams.go#L254 Note: If the query is used somewhere else we'd need to make the order of the...

No, no reason! I was at a conference and didn't find a minute to review this yet :) Am Mi., 2. Nov. 2022 um 10:15 Uhr schrieb Kaffee☕ ***@***.*** >:...

The streams themselves are marked as `Live` whereas the courses that are currently live don't have information about the date of their next livestream (so it says `currently live` instead)....