-
Notifications
You must be signed in to change notification settings - Fork 7
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
Convert to REST APIs. #22
Comments
Issue-Label Bot is automatically applying the label Links: app homepage, dashboard and code for this bot. |
Hello, I would be interested in taking on this issue for Hacktoberfest. I would appreciate being assigned if this is ok with you. Thank you! I would also need more info about what needs to be included in the REST api. |
@fisher60 sure. You can go ahead and start working on it. Please note: Incase of no activity for a long time, the project will be assigned to someone else. @raysandeep will be briefing you here what all are needed for the REST API. |
Alright, sounds good. I would appreciate the brief whenever possible. Thank you. |
Hey @fisher60,
For the first one, the algo will be available in the dashboard route handler you can use that. Pass on the location f file it will give you the output for the same. For the second one, you need to basically flush out all the values in DB by filtering out with user id. Remember to make these both routes you need to have some Rest API authentication. I would recommend you to use Token Auth. Check out this library Djoser. Feel free to reply in this thread if you're stuck. I would love to help you out. ✌ |
I will need help setting up this project. I am getting errors when installing the requirements to virtualenv. The readme instructions say to use global installs, but I am not going to do that. |
@fisher60 that’s a typo. Good catch. Scripts are in the env folder. You can correct this in your PR. |
The current code uses Django templating. Try converting the project into REST APIs with proper documentation.
Brownie points if you can link the APIs too.
The text was updated successfully, but these errors were encountered: