TEAMMATES
Copying feedback session: Name for copied session should not be whitespace
Steps to reproduce
- Head to Sessions page.
- On any course, click on Copy.
- You should see this that the
Name for copied session*has the same name as the Session you clicked Copy on. Now, clear the text input and this is what you will get below.
Now, let's try to add a whitespace. Validation passes i.e., There is a value for the copied session's name and it is a whitespace.
Expected behaviour
Whitespace should not be allowed for the copied session's name. Input validation should not allow whitespaces.
Actual behaviour
Converse of the above expected behaviour, i.e., Currently, whitespace as name is allowed and this happens after clicking on Copy:
This is validated on the client side. However, we should validate on the form first.
Additional info
Motivation for this issue from a PR: https://github.com/TEAMMATES/teammates/pull/12670#issuecomment-1868207324
Proposed Solution:
- You may look at this link on form validation: https://www.javatpoint.com/angular-form-validation-no-whitespace-allowed
- Refer to this file in the codebase:
src\web\app\pages-help\session-links-recovery\session-links-recovery-page.component.ts, there is an example on how validation is done L39, and you may use the above link in Point 1 to help you too. And this is the page (http://localhost:4200/web/front/help/session-links-recovery):To get to the above page, once you're logged in, in the navbar, click on Help -> Recover Session Links -> Hit on Submit without any input.
Let us know if you have any question.
Good First Issue - Notes for Contributors This issue is for first-time contributors only. If you are new to TEAMMATES, feel free to submit a PR for this issue.
Please note that we allow only one good first issue per contributor. If you have already made a prior contribution to TEAMMATES, you may wish to take a look at issues with the help wanted tag instead.
We do not assign issues to contributors. If you would like to pick up this issue, do post a comment below to express your interest and check if there is anyone else who is already working on the issue. We will do our best to reply and give you the go-ahead, but if we don't, feel free to submit a PR as long as there is no one else working on it.
To get started, do read through our contributing guidelines carefully, and set up a development environment on your local machine before making a PR.
If you need any clarifications on our developer guide, or are facing issues that are not found in our troubleshooting guide, please post a message in our discussion forum.
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
@rahulpro2001 Hello, please allow @dhanushtheijas08 to work on it first as he has expressed in interest. Feel free to look at other issues tagged with good first issue for the time being!
can i work on these issue please, these is my first open source contribution !
Hi Could any one help me setting up project in my local system, I am getting error at build.properties as null pointer exception. Thanks
can i work on these issue please, these is my first open source contribution !
hi @ArunErram please allow @dhanushtheijas08 to work on it first as he has expressed interest for this issue
hi @pavanganguru, do open a help request here: https://github.com/TEAMMATES/teammates/discussions, it would be good to provide screenshots or more details about the error you're facing as well
I see that this issue is still open, can I work on it? Also, how can I become a contributor for this repo?
Hey! is the issue resolved? If it is still open , i would like to contribute to this issue! Please let me know if I can
@dhanushtheijas08 Hello, is there any update on this issue? Else, I will ping the next person to attempt it. Thanks!
Hey.I still see this issue open and showing not resolved in my local.Can I take it up?
@rahulpro2001 Please feel free to submit a PR for this if you're still interested. Else, I will pass it onto the next person. Thank you!
@Afreen97 Let's go down the queue. Feel free to look at other issues tagged good first issue for the time being!
hey @domlimm, i'm also interested in working on this. please add me to the queue.
@ArunErram @IvoHarsani @michaelviveiros Do let me know if any of you wants to take up this issue.
@ArunErram @IvoHarsani @michaelviveiros Do let me know if any of you wants to take up this issue.
@domlimm If it's available I would like it.
@ritsam Hello, lets give @michaelviveiros some time to work on this. Michael do let us know if you're not working on it anymore.
@ritsam in the meantime, you may look for other good first issue to work on.
i think, i missed the oppurtunity , do i have the chance now to work on it @domlimm ?
hi @michaelviveiros, do let us know within 2 days if you're still working on this issue, else we'll let @ArunErram work on it
Hi @ArunErram, since @michaelviveiros hasn't replied, please feel free to submit a PR for this issue. Thanks for your patience!
@weiquu @domlimm i have started working on it . But while i am trying to login as student or instructor the login page is not getting displayed .
and from where can i get the test credentials to login and test the expected behaviour after making the changes?
and from where can i get the test credentials to login and test the expected behaviour after making the changes?
Hi @ArunErram, do refer to https://teammates.github.io/teammates/development.html#logging-in-to-a-teammates-instance for logging in to the local dev environment