mentorship-backend
mentorship-backend copied to clipboard
anitab-org
Add query parameter to filter users by verification state to GET /users API
Description
As a developer,
I need a filter feature on the Users listing endpoint,
so that I can reuse GET /users API to filter users who have the email verified.
GET /users API needs a way to filter users by email verification state (which is represented by a boolean value). This will allow us deprecating the extra GET /users/verified API.
Acceptance Criteria
Update [Required]
- [ ] Add a query parameter support to
GET /users, to filter by its current state. - [ ] Update Swagger UI documentation for this API.
- [ ] Write at least one test to prove your new feature is working.
Definition of Done
- [ ] All of the required items are completed.
- [ ] Approval by 1 mentor.
Estimation
4 hours
Yes you can @Juhi-Purswani :) Be aware that sometimes you may not see someone assigned, but if you see in the comments, someone claiming it and a maintainer approving it, it can mean the contributor is working on it. No one has claimed this yet, so feel free to work on this and let me know if you have any doubt
Sorry for the delay in my response @lorryaze I think you are already occupied with other issues, so I will keep this available. Also because there are no updates from @Juhi-Purswani
@isabelcosta apologies for the dealy, wasn't able to work due to some reasons. But i have worked on the issue, will submit the PR in a day
@Juhi-Purswani are you still working on this? 9 days have passed and you haven't created a PR still. I have no problem with that :) it's better if you update us if you can or cannot work on this than to block the issue for you and not letting other contributors working on this.
We just want some transparency. If you cannot work on this let us know so we can assign it to another contributor.
@isabelcosta due to some socio-political reasons there is too much network issue in my area. I may not be able to work for coming 1 week more. Please assign the issue to any other contributor.
Thank you so much for the update @Juhi-Purswani . I'm sorry to know about that. I will leave this specific issue available for other contributors then. Although, I will keep creating issues for first timers so once you are able to contribute you will have issues ;)
Hi @isabelcosta,
Can you please assign this to me? I will submit the PR in 1-2 days. Thanks.
Hi @isabelcosta,
Can you please assign this to me? I will submit the PR in 1-2 days. Thanks.
I was trying to setup the project but I am getting some errors on my system. I will let you know.
Hi @isabelcosta, Can you please assign this to me? I will submit the PR in 1-2 days. Thanks.
I was trying to setup the project but I am getting some errors on my system. I will let you know.
I am getting errors for pypiwin32==200
I am using any random values for the environment variable SECRET_KEY and SECURITY_PASSWORD_SALT and dev for FLASK_ENVIRONMENT_CONFIG. Is this setting ok?
Hi @isabelcosta, Can you please assign this to me? I will submit the PR in 1-2 days. Thanks.
I was trying to setup the project but I am getting some errors on my system. I will let you know.
I am getting errors for pypiwin32==200
I also tried with docker but :(
@Ashokrayal could you check this topic on Zulip to see if anyone went through the same issue as you? Could you also post your problem there, in that way most of the community will be aware and can also help you.
Hi @isabelcosta. I am new to open source. Can I work on this issue?
@Juhi-Purswani are you still interested on working on this issue? Please let me know so that I can know to whom I should assign this.
If you feel you won't be able to work on this, please let us know. I will have to assign this to someone else within the next 24h if you aren't available to work on this. There are plenty of issues available you can work on, when you become available.
@satya7289 are you still interested in contributing to this PR? If yes please let me know so I can assign it to you.
Ok I jsut noticed you are assigned to another issue @satya7289 . @cs17b005 are you interested in this issue? I can assign this to you if you are.
Yeah! I am interested. But someone else told they have already submitted a pull request?
According to our Community Guidelines, you can only submit a pull request once issue is approved and you are assigned to it. I will assign this to you, as I checked the order of requests to work on this @cs17b005
@Ashokrayal can you please look into other issues, there are plenty here :)
I am working on it. But when I try to run it I get the following error. Could you help me solve it? sqlalchemy.exc.NoSuchModuleError
Hey @cs17b005 I am not very good in flask, but I too came through this error when I started with the language. I referred to this to solve the issue Hope this helps.
I have referred to it and many of them say it was caused due to a typing error in spelling of sqllite but I didn't find that anywhere in this code. @Ashokrayal could you please tell me how you solved the errors you got in your machine and also can you help me on this one?
@isabelcosta Yes I am still interested and I will be available to work but since this issue has already been assigned I will search for some other issues. Thanks.
@Juhi-Purswani oh ok! I am sure we have plenty of issues for you to work on, let me know if you need any help.
@cs17b005 I have commented few packages in requirement file which were related to docker. I found this solution on tulip app.