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

Modifier la période d'obscolence par défaut pour certains logins #869

Open
MathieuDomingo opened this issue May 26, 2023 · 5 comments
Open

Comments

@MathieuDomingo
Copy link

Salut,

l'idée serait de pouvoir indiquer les logins de certains utilisateurs pour outrepasser la valeur par défaut obtenue via l'affiliation :

ACCOMMODATION_YEARS={'faculty': 3, 'employee': 3, 'staff': 3}
=>
ACCOMMODATION_YEARS={'faculty': 3, 'employee': 3, 'staff': 3, 'MonLogin': 5}

Et niveau code il faudrait juste modifier la fonction save du fichier podv2/pod/video/models.py en rajoutant le 2e if :

...
            # fix date_delete depends of owner affiliation
            ACCOMMODATION_YEARS = getattr(settings, "ACCOMMODATION_YEARS", {})
            if ACCOMMODATION_YEARS.get(self.owner.owner.affiliation):
                new_year = ACCOMMODATION_YEARS[self.owner.owner.affiliation]
                self.date_delete = date(
                    date.today().year + new_year,
                    date.today().month,
                    date.today().day,
                )
            if ACCOMMODATION_YEARS.get(self.owner.username):
                new_year = ACCOMMODATION_YEARS[self.owner.username]
                self.date_delete = date(
                    date.today().year + new_year,
                    date.today().month,
                    date.today().day,
                )
...

Bonne journée,
Mathieu

@ptitloup
Copy link
Contributor

Bonjour Mathieu,
Pour information le prochain atelier du GT ESUP dématérialisation aura lieu le mardi 20 juin prochain de 9h à 12h00 en distanciel. Il portera sur le sujet de l'archivage numérique avec Pod comme exemple.
J'en ferai la promo sur le canal pod des que j'aurai recu le mail "officiel".
Bonne journée
Nicolas

@ptitloup
Copy link
Contributor

Et pour érpondre à ta proposition, je partirai plutot sur un groupe que sur un individu ;)

@MathieuDomingo
Copy link
Author

Salut Nicolas,

Je suis parti sur individu parce que :

  • a priori je n'ai que 2 utilisateurs qui ont besoin d'un traitement spécifique.
  • le login est unique, c'est donc la même simplicité de traitement que l'affiliation principale.

Si on utilise des groupes, on perd la simplicité induite par l'unicité, et il faut donc établir des règles pour savoir ce que l'on fait pour les utilisateurs qui appartiennent à plusieurs groupes :-/ (de base sans trop réfléchir j'aurai tendance à dire qu'on garde la plus grande valeur par défaut mais il y a peut être des cas où ce n'est pas le comportement voulu ?)

On doit pouvoir envisager de combiner les 3 (a priori avec une priorité lié à la généricité : affiliation < groupe < login)

J'ai voulu faire quelques petits tests à tatons pour les groupes, mais je ne sais pas comment on y accède 😅
for group in self.owner.owner.groups.all():

@ptitloup
Copy link
Contributor

ptitloup commented Jun 6, 2023

Hello,
Alors cette migration en V3, ca avance ?
Sinon, oui il faudrait partir sur un group, sinon, tu vas devoir redemarrer ton Pod à chaque ajout d'individu.
On partirait bien sur les groupes de gestions (groups) si on suit ton idée. Je vais en parler en Copil et en Cotech, je te tiens informé
@+
Nicolas

@MathieuDomingo
Copy link
Author

C'est en phase de test, bascule pour cet été normalement 👍

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

No branches or pull requests

2 participants