teammates
teammates copied to clipboard
TEAMMATES
Instructor creating courses: auto-trim text fields
v8.19.2, production
Current: When creating a course, there will be an error message if the course ID or the name has leading/trailing white spaces
Suggestion: Auto-trim those fields as such leading/trailing spaces are sure to be accidental rather than intended.
@domlimm let's leave this to a first time contributor (unless someone from the dev team thinks it is too much for a first timer)
Lets make it first come first serve. @thomasengels Would you like to proceed with it?
I will check the code later today. Just have been super busy with my actual developer job
Hello, I would like to work this issue. Can anyone tell me which file does this "COURSES" exist ??
Hello, I would like to work this issue. Can anyone tell me which file does this "COURSES" exist ??
I couldn't find it either. I did a text search for words on the page but found numerous files containing it. Not sure what we'd edit.
@thomasengels Any updates?
@akhil9281 @seb-lex Hold on first. Someone has taken it and yet to give up. It would probably not be nice to post duplicated PR on the same issue. If you would still like to investigate the issue, for the handler, I would suggest to trace down by finding the correct servlet class from this file: https://github.com/TEAMMATES/teammates/blob/675374990ed027b474f63fc40ccfd7524ff0baaf/src/main/webapp/WEB-INF/web.xml
Hey is the issue still open? . I would like to take this on . my first contribution to opensource
I think it can be considered that @thomasengels gave up on this since no replies has been received.
@Basfar @Ranzeb @Shubhcs01 @akhil9281 @seb-lex Any of you would like to take a look at this? If not, will pass to @LokeshSingh1102 .
Is this issue open? looking for my first contribution to open source projects.
I think they are not taking this on. Should i proceed with this issue @fsgmhoward??
I tried to build the back-end server, in Git Bash using both ./gradlew serverRun & and ./gradlew.bat serverRun &. The results are the same: I never got to "BUILD SUCCESSFUL". Instead, the console outputs:
<===========--> 85% EXECUTING [45m 48s]
The time in the brackets keep incrementing, even if I leave the console overnight. The time gets updated but it never gets past 85%. Can you give me some pointers to move forward?
I'm trying to fix this issue. I would like to first replicate the issue, I believe I need instructor privilege in order to create a course. Since I'm not an instructor, what can I do to gain sufficient privilege for issue replication?
Hi @ShubhraSuman there is no open PR for this issue, would you like to work on it?
@damithc @fsgmhoward may I work on this issue? I can do it immediately.
My idea of doing it is to let sanitizeForSaving() in src/main/java/teammates/common/datatransfer/attributes/CourseAttributes.java to trim the course name. This can be done by adding only one line of code. Also, trimming invalid names into valid should be the job of sanitizeForSaving() and this approach is aligned with the origianl structure of the code.
I are doing a group assignment at uni that has to work on an issue from open-source software. We hope to contribute as much as possible as the due date is very soon. I will greatly appreciate it if you allow me to work on this issue.
@fsgmhoward @ypinhsuan Sorry accidentally open a PR. I think I should contact you and get your approval before working on this issue. If you allow me to do this issue, I will open a pull request again.
Hi @Haoting-Chen , the person taking this issue previously has not given up. Please seek other issues to work on. Thank you so much.
@ShubhraSuman Sorry for bothering you. I am currently working on a group assignment at uni that we have to solve as many issues as we can for Teammates. The assignment is due next week. If you haven't started yet, would you like to hand over this issue to me? I have figured out how to do it and can start immediately.