-
Notifications
You must be signed in to change notification settings - Fork 448
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add to List Users API the ability to filter users by skills #759
Comments
Hi, I want to work on this one. please assign it to me. |
@ashokrayal Another issue has been assigned to you. Please complete that and check back. |
Hi @paritoshsinghrahar, |
@Manav-03 I have assigned you. I hope you have set up the environment and read community guidelines. Happy Coding. 😃 |
@paritoshsinghrahar Thank you, I'll get to it right away. :) |
@Manav-03 how's the PR going? |
@vj-codes I had some trouble setting the environment up but I'm working on it right now :) |
@Manav-03 if the errors are solved, share them here or on zulip, we could help you then:) |
@vj-codes I set up the environment and installed the requirements. However, when I tried to run 'run.py' it gave me some errors about not having the module pysqlite. I tried to install pysqlite but it didn't work as python 3 does not support pysqlite, it said that that I should install pysqlite3 instead, which I did, but it hasn't helped. |
@Manav-03 please can you provide screenshot of - error and your .env file |
@vj-codes |
@Manav-03 secret_key should be different from security_password_salt , either update any one to Mostly the error was due to the .env file , update what I mentioned above and see if it runs |
@Manav-03 remove the current virtual environment and name it as |
I made these changed to .env and the problem still persists. Should I delete the virtual environment I created and create another one with the name venv? |
@Manav-03 I would say yes |
Is this still available if yes then can i take this up ..Thanks. |
@pallavisavant The issue is assigned to me. I've had some issues on my system while working on it, due to which I've been slowed down. If I can't figure it out in a few more days I will unassign myself. |
@Manav-03 ok no issues....Thanks. |
@Manav-03 what do you see on your localhost now? |
@vj-codes Ever since I removed the environment that I first created to make another one with the appropriate name my terminal is not even recognizing 'Python', I tried to add it to the PATH variable but I'm still facing issues. I'll figure out the issues in my system ASAP and get back to it. |
Hi @Manav-03 , Are you still working on it? |
Hi @ashokrayal, I've been facing a lot of issues with my system, I'll just un-assign myself. |
@Manav-03 it's ok, If you want to take more time. I just wanted to know if you are still working on it. I will be happy be help you in your issues. |
I also step up the project on my system recently. |
@ashokrayal I've tried all of that but the issue just seems to be something with my system and it's taking me a lot of time to figure it out. I think it's best if I try to figure that out instead of keeping this issue unavailable for people who could actually work on it. I'll take up another issue once I've sorted the issues with my system out :) |
Hi @vj-codes / @paritoshsinghrahar , I would like to work on this issue. I have no issues in progress. I already asked the same on 29th Aug. Thanks |
Assigning you @ashokrayal |
thanks @vj-codes. |
Hi @vj-codes. Is this issue open? If it is, then I would like to work on this. |
Assigning you @battuAshita |
Hi @vj-codes. While working on this issue, initially, the server was not able to connect to the database config through the .env file. So, I have added the line Also, is there a condition that each user can have more than one skill? |
No need |
@isabelcosta thoughts please. |
Hi @vj-codes. I was running my test case but was unable to do it. I am getting the following error. |
@battuAshita heyy are you using FLASK_ENVIRONMENT_CONFIG as local and you should not have the db variables in your .env file |
Yes, I am using it as local. |
This issue has been resolved. Thank you. |
I have raised a PR. Please review it :) |
Description
As a member of the app,
I need be able to filter other members by their skills,
so that I can choose the member I want to create a mentorship relation with.
APIs used to fetch Users, don't have an option to filter users by
skills
attribute.Note: This issue is labeled for Open Source Hack.
I can assign this issue to you on September 1st as the official opening date for Open Source Hack. Please free to check out other issues in the meantime. Program website: https://anitab-org.github.io/events/open-source-hack/
Acceptance Criteria
Update [Required]
GET /users
andGET /users/verified
, to filter users byskills
attribute.Definition of Done
Estimation
[4] hours
The text was updated successfully, but these errors were encountered: