✨ [#315] added extra version and uuid fields in admin #393
Conversation
bart-maykin
force-pushed
the
feature/315-extra-admin-fields
branch
2 times, most recently
from
4afd616 to
d296fc6
Compare
src/objects/core/models.py
Outdated
| @@ -40,6 +40,17 @@ def url(self): | |||
| # zds_client.get_operation_url() can be used here but it increases HTTP overhead | |||
| return f"{self.service.api_root}objecttypes/{self.uuid}" | |||
|
|
|||
| @property | |||
| def version(self): | |||
There was a problem hiding this comment.
It's not a version, it's rather the last available version. But the thing is that we specify the objecttype version in the object data, Objecttype model has only a link to objecttype itself.
@joeribekker Do we really want in the list admin view to perform a lot of HTTP requests to get the last version?
Even if you want to go this was, I think it's more useful to show the list of all versions available, not the last one.
And if you want to see them in the list view, it's better to perform one list request to Objecttypes API instead of a lot of individual detail requests
There was a problem hiding this comment.
I think I specced this ticket too easily. And we dont want to do many http calls here. I will respec the ticket.
bart-maykin
force-pushed
the
feature/315-extra-admin-fields
branch
from
2207eab to
01ac34a
Compare
bart-maykin
force-pushed
the
feature/315-extra-admin-fields
branch
from
01ac34a to
c3057d2
Compare
bart-maykin
force-pushed
the
feature/315-extra-admin-fields
branch
from
c3057d2 to
517c8d4
Compare
bart-maykin
force-pushed
the
feature/315-extra-admin-fields
branch
from
517c8d4 to
491d1ab
Compare
Fixes #315
Changes
Added some version and uuid fields in the admin panel.