entrouvert
/
lingo
15
0
Fork 0
Code Issues Pull Requests 8 Releases Wiki Activity

manager: display applications (#86636) #162

Merged
lguerin merged 1 commits from wip/86636-applification-in-ui into main 2024-03-15 17:14:41 +01:00
Conversation 1 Commits 1 Files Changed 29 +917 -127
lguerin commented 2024-02-08 15:34:59 +01:00
Owner
Copy Link
No description provided.
lguerin added 1 commit 2024-02-08 15:35:00 +01:00
gitea/lingo/pipeline/head This commit looks good Details
25eadefb47
manager: display applications (#86636)
lguerin force-pushed wip/86636-applification-in-ui from 25eadefb47 to 9988f5fb7c 2024-02-09 08:35:21 +01:00 Compare
pmarillonnet requested review from pmarillonnet 2024-02-27 15:39:04 +01:00
pmarillonnet commented 2024-02-28 10:04:33 +01:00
Owner
Copy Link

(Je commence à relire.)

(Je commence à relire.)
lguerin changed target branch from wip/86291-application to main 2024-02-29 12:07:59 +01:00
lguerin force-pushed wip/86636-applification-in-ui from 9988f5fb7c to 0253fed31e 2024-02-29 12:09:10 +01:00 Compare
pmarillonnet reviewed 2024-03-14 09:50:20 +01:00
pmarillonnet left a comment
Owner
Copy Link

Deux mini trucs relevés (edit: en fait un seul truc) au passage, plus pour ma compréhension propre qu’une remise en question de la PR.

Deux mini trucs relevés (edit: en fait un seul truc) au passage, plus pour ma compréhension propre qu’une remise en question de la PR.
lingo/export_import/models.py Outdated
@ -23,1 +23,4 @@
class WithApplicationMixin:
@property
def applications(self):
pmarillonnet commented 2024-03-14 09:30:26 +01:00
Owner
Copy Link

Ok, j’ai loupé un bout de l’UI applification je pense alors je ne suis pas sûr de comprendre : je vois plus bas cette propriété utilisée dans les templates, mais c’est dans quels cas qu’on a intérêt à consulter les applications dont l’objet fait partie plutôt que le (a priori plus intuitif) inverse à savoir les objets contenus dans une application ?

Edit: Ah, ok, pigé dans la suite de la relecture, on affiche l’info de la liste des applications concernées pour chaque objet concerné dans le BO.

Ok, j’ai loupé un bout de l’UI applification je pense alors je ne suis pas sûr de comprendre : je vois plus bas cette propriété utilisée dans les templates, mais c’est dans quels cas qu’on a intérêt à consulter les applications dont l’objet fait partie plutôt que le (a priori plus intuitif) inverse à savoir les objets contenus dans une application ? Edit: Ah, ok, pigé dans la suite de la relecture, on affiche l’info de la liste des applications concernées pour chaque objet concerné dans le BO.
pmarillonnet marked this conversation as resolved
lingo/export_import/models.py Outdated
@ -109,0 +106,4 @@
@classmethod
def get_orphan_objects_for_object_class(cls, object_class):
content_type = ContentType.objects.get_for_model(object_class)
elements = ApplicationElement.objects.filter(content_type=content_type, application__visible=True)
pmarillonnet commented 2024-03-14 09:46:23 +01:00
Owner
Copy Link

Pour quelle raison ici la recherche des objets orphelins se soucie de application__visible=True alors que la méthode similaire get_objects_for_object_class (juste au dessus) ne prend pas en compte ce flag visible ?

Pour quelle raison ici la recherche des objets orphelins se soucie de `application__visible=True` alors que la méthode similaire `get_objects_for_object_class` (juste au dessus) ne prend pas en compte ce flag `visible` ?
lguerin commented 2024-03-14 10:29:43 +01:00
Author
Owner
Copy Link

get_objects_for_object_class s'applique à une Application déjà connue (on l'a dans l'url), et visible:


        if 'application' in self.request.GET:
            self.application = get_object_or_404(
                Application, slug=self.request.GET['application'], visible=True
            )

(cf WithApplicationsMixin)

get_orphan_objects_for_object_classcherche les objets ne faisant partie d'aucune application visible

`get_objects_for_object_class` s'applique à une Application déjà connue (on l'a dans l'url), et visible: ``` if 'application' in self.request.GET: self.application = get_object_or_404( Application, slug=self.request.GET['application'], visible=True ) ``` (cf WithApplicationsMixin) `get_orphan_objects_for_object_class`cherche les objets ne faisant partie d'aucune application visible
pmarillonnet commented 2024-03-15 16:55:35 +01:00
Owner
Copy Link

Hmm, okay, j’avais loupé ça, merci pour l’explication.

Hmm, okay, j’avais loupé ça, merci pour l’explication.
pmarillonnet marked this conversation as resolved
lguerin force-pushed wip/86636-applification-in-ui from 0253fed31e to f82468d154 2024-03-15 08:42:23 +01:00 Compare
pmarillonnet approved these changes 2024-03-15 16:55:51 +01:00
lguerin merged commit 26200949cc into main 2024-03-15 17:14:41 +01:00
lguerin deleted branch wip/86636-applification-in-ui 2024-03-15 17:14:41 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
pmarillonnet
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/lingo#162
No description provided.
Delete Branch "wip/86636-applification-in-ui"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?