FergusMok
FergusMok
@tau-bar Thanks for the work so far! Definitely see the codebase being cleaner now
@tau-bar I'm ok with the PR after the last change and fixing the E2E tests! Will wait for one more review
@tau-bar Just a small reminder about the PR!
I was originally thinking that it should be closed on default, so that the user can easily see the 2 buttons at the bottom and understand the workflow more easily....
This may be somewhat related, but the better solution may be to just lint files that have diff. Currently, `npm run lint` lints all typescript/html files, which is unnecessarily long....
Perhaps the default value (e.g 10%) can be built into the component too, so that all the progress bars can benefit e.g for the edit course progress bar. May be...
Hi @samuelfangjw @jayasting98 do you have any opinions on the solutions? Hopefully we can work something out before the announcement of the feature
@tau-bar Yes please go ahead :)
Hi @Alc-Alc I've tried reproducing using the MCVE on `804f015`, but I'm still somehow able to get the entirety of the logs. I've tried using Python 3.12.0 and 3.12.1. ```...
@cedricongjh Added the deadline extension Awaiting confirmation about the field length, and will be adding changelog for beta 7 to the PR once confirmed