[API] Endpoint for adding/updating user info

[Aksh-Bansal-dev]

No description provided.

[Samy-33]

@PranshuNayak, I am removing this assignment as there are few others assigned to you which you can work on and others can take these tasks up.

[arminpatel]

I'm working on this.

[arminpatel]

I'm a little confused which model is to be serialized for the endpoint.
Should I create a serializer for only User model or for all of them (Student, Faculty and Staff)?
Or should I create serializer for Student, Faculty and Staff and add extra fields to them (like name, email etc)?, imo this seems better.

[Samy-33]

How about creating a serializer for User, and have three fields in it for student, faculty, staff.

This is nested serialization: https://www.django-rest-framework.org/api-guide/relations/#nested-relationships

[arminpatel]

How about creating a serializer for User, and have three fields in it for student, faculty, staff.

This is nested serialization: https://www.django-rest-framework.org/api-guide/relations/#nested-relationships

Exactly what I wanted. Thanks.

[arminpatel]

Does this endpoint also creates users?

[Aksh-Bansal-dev]

Yeah, you can create 2 endpoints one for updating and one for creating.

[Samy-33]

@arminpatel raise 2 separate CLs. One for Create and another for Update

[arminpatel]

raise 2 separate CLs

what are CLs? 😅

[Samy-33]

what are CLs? 😅

Sorry, got confused between internal terminology and external. It's called a changelist same as pull request.

So raise two PRs separately for update and create.

[arminpatel]

Who will have permission to update user?

[Aksh-Bansal-dev]

Who will have permission to update user?

I think only the user should have permission to edit their profile details.

[Samy-33]

I think only the user should have permission to edit their profile details.

Admin user should have permission to edit user information as well.