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

[OS-9314] Create a single view and change the css class of the thread view #17

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

jcougnaud
Copy link

No description provided.

@jcougnaud jcougnaud force-pushed the feature/OSIS-9314 branch 2 times, most recently from 9b6cf35 to 193ff0f Compare October 22, 2024 14:00
@jcougnaud jcougnaud changed the base branch from dev to 0.17 October 22, 2024 14:06
@jcougnaud jcougnaud changed the base branch from 0.17 to dev October 22, 2024 14:06
@albrugnetti
Copy link
Contributor

@jcougnaud Est-ce qu'il ne faut pas de vue utilitaire Django ?

@jcougnaud
Copy link
Author

@jcougnaud Est-ce qu'il ne faut pas de vue utilitaire Django ?

@albrugnetti J'ai l'impression que le fonctionnement pour la vue simple était déjà pris en compte dans la mixin existante :

if 'last' in self.request.GET:
alors je l'ai simplement réutilisée. A voir si cela convient d'uniformiser le comportement pour les deux vues comme c'est le cas actuellement ou si on préfère les distinguer.

@albrugnetti
Copy link
Contributor

@jcougnaud Est-ce qu'il ne faut pas de vue utilitaire Django ?

@albrugnetti J'ai l'impression que le fonctionnement pour la vue simple était déjà pris en compte dans la mixin existante :

if 'last' in self.request.GET:

alors je l'ai simplement réutilisée. A voir si cela convient d'uniformiser le comportement pour les deux vues comme c'est le cas actuellement ou si on préfère les distinguer.

C'est préférable d'avoir 1 seul vue :D ! Donc on laisse en l'état ! Merci

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants