ls1intum / Hephaestus

Hephaestus - A Framework to Support Novice Software Engineers in Collaborative Environments
https://hephaestus.ase.cit.tum.de
MIT License
8 stars 1 forks source link

Fix: Use correct After-queryParam when none specified #188

Closed GODrums closed 1 week ago

GODrums commented 1 week ago

Motivation

Fixes an issue where the after param is after the before param due to the week starting on Tuesdays. I.e. https://hephaestus.ase.cit.tum.de redirects to https://hephaestus.ase.cit.tum.de/?after=2024-11-26T09:00:00%2B01:00&before=2024-11-25T00:22:47%2B01:00 right now, making the default leaderboard data incorrect.

Description

This fix moves the default date configuration to the leaderboard filter component, making sure dates are only changed in one place of the application. Also fixes a small bug regarding the recently merged PR #124, where the default value of the Team-Color-Input had an invalid default value: Screenshot 2024-11-25 000613

Testing Instructions

Screenshots (if applicable)

Checklist

General