Skip to content
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

Simplify the API #107

Open
francescarpi opened this issue Oct 20, 2020 · 5 comments
Open

Simplify the API #107

francescarpi opened this issue Oct 20, 2020 · 5 comments
Assignees
Labels
backend Dev team enhancement New feature or request
Milestone

Comments

@francescarpi
Copy link

Is your feature request related to a problem? Please describe.
The current API is complex and unusable. Swagger is here:

http://nemperfeina.cat/api/swagger

Describe the solution you'd like
Simplify only by keeping the following services:

  • /job/ (CRUD)
  • /user/ (CRUD)
  • /login/

Other services can be removed.

Describe alternatives you've considered
No

Additional context
No

@francescarpi francescarpi added backend Dev team enhancement New feature or request labels Oct 20, 2020
@fullonic
Copy link

@wolframtheta Before seeing that you has assigned I started working on /jobs. I'm just missing improve the tests. Have you done already some work on jobs? If I can help, let me know.

@wolframtheta
Copy link

wolframtheta commented Oct 22, 2020

@wolframtheta Before seeing that you has assigned I started working on /jobs. I'm just missing improve the tests. Have you done already some work on jobs? If I can help, let me know.

@fullonic Nop, I haven't started yet the jobs endpoint, because I'm not used to use Django and I'm learning how it is organized each endpoint

@victormartingarcia
Copy link

Just FYI I'll start to work on simplifying the model (#108), it might affect the API simplification as well so we should work closely ;)

@fullonic fullonic self-assigned this Oct 25, 2020
@francescarpi francescarpi added this to the sprint 4 milestone Oct 27, 2020
@francescarpi
Copy link
Author

Hi! This issue can be closed? The api simplification it's already done in #108?
Thanks!

@wolframtheta
Copy link

wolframtheta commented Oct 29, 2020

I do not see in the commits that the login endpoints part have been modified, isn't it? And the same for the user endpoints.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Dev team enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants