API: Récupérer des WasteActions avec leur ResourceActions (si l'utilisateur est connecté) #4544
Conversation
raphodn
force-pushed
the
raphodn/backend-resource-action-create
branch
from
e536a78
to
3f9d008
Compare
raphodn
force-pushed
the
raphodn/backend-resource-get-with-actions
branch
from
69a04c9
to
5261f51
Compare
raphodn
force-pushed
the
raphodn/backend-resource-get-with-actions
branch
from
5261f51
to
b38801c
Compare
raphodn
force-pushed
the
raphodn/backend-resource-get-with-actions
branch
from
b38801c
to
469dfe4
Compare
| @@ -39,3 +39,10 @@ class WasteActionView(RetrieveAPIView): | |||
| model = WasteAction | |||
| queryset = WasteAction.objects.all() | |||
| serializer_class = WasteActionSerializer | |||
|
|
|||
| def get_serializer(self, *args, **kwargs): | |||
| if self.request.user.is_authenticated: | |||
There was a problem hiding this comment.
vu qu'on a besoin de passer le user au serializer, je n'ai pas trouvé mieux que les kwargs (j'ai vu que c'était déjà utilisé ailleurs (canteen, "action"))
There was a problem hiding this comment.
est-ce que c'est possible de passer actions directe ? C'est-à-dire mettre la logique de user et queryset qui est maintenant dans le serializer ici ?
There was a problem hiding this comment.
j'ai essayé mais je n'arrive pas à filtrer le queryset de la FK.
ici dans le vue ca permet seulement de filtrer le queryset de WasteAction, pas ResourceAction.
There was a problem hiding this comment.
mais en fait ca bloque sur le fait que je n'arrive pas à passer un queryset custom à serializer custom 🤔
le queryset custom dans WasteActionWithActionsSerializer pour le champ actions est ignoré..
There was a problem hiding this comment.
ok je crois que j'ai trouvé une solution avec SerializerMethodField
data/models/resourceaction.py
Outdated
| @@ -5,12 +5,19 @@ | |||
| from .wasteaction import WasteAction | |||
|
|
|||
|
|
|||
| class ResourceActionQuerySet(models.QuerySet): | |||
| def for_user(self, user): | |||
There was a problem hiding this comment.
si vous trouvez un meilleur nom de for_user let me know ^^
There was a problem hiding this comment.
for_managed_canteens/for_user_canteens ?
| self.assertTrue("actions" in response.data) | ||
| self.assertEqual(len(response.data["actions"]), 1) | ||
| self.assertEqual(response.data["actions"][0]["canteen_id"], self.canteen.id) | ||
| self.assertEqual(response.data["actions"][0]["canteen"]["id"], self.canteen.id) |
There was a problem hiding this comment.
pourquoi avoir les deux canteen_id et canteen: {id} ?
There was a problem hiding this comment.
le serializer ResourceActionWithCanteenSerializer hérite de ResourceActionSerializer du coup il a les deux (l'id, et l'objet). ca ne me choque pas ^^
raphodn
force-pushed
the
raphodn/backend-resource-get-with-actions
branch
from
7d0dd34
to
cd8b85a
Compare
Quoi ?
Suite de #4500
On améliore l'endpoint API existant
wasteActions/<int:pk>, en rajoutant un champactionssi l'utilisateur est connecté.Ce champ contient une liste des ResourceAction existantes pour les cantines auxquelles l'utilisateur appartient.
Pour cela