gocast
gocast copied to clipboard
TUM-Dev
reverse course order
Description
Change the order planned or upcoming courses are shown.
Motivation and Context
Usually newest dates are displayed first instead of last.
Implementation: Simply reverse the array that holds the information about given upcoming or planned courses
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 methods result configurable
Is there a reason this isn't merged yet? The query isn't used anywhere else so there is no need for configuration.
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☕ @.***
:
Is there a reason this isn't merged yet? The query isn't used anywhere else so there is no need for configuration.
— Reply to this email directly, view it on GitHub https://github.com/joschahenningsen/TUM-Live/pull/731#issuecomment-1299914955, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKV24QF2WIEB6625WPB47KDWGIWJNANCNFSM6AAAAAARJCR5LI . You are receiving this because you commented.Message ID: @.***>