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

[DONE] Pampletousse/fix accomodation years date delete #921

Conversation

pampletousse
Copy link
Collaborator

Fix de la date de suppression non calquée sur les années d'obsolescences par affiliation définies dans ACCOMMODATION_YEARS. A cause de users dont la value affiliation est parfois une string, parfois un tableau de valeur

ptitloup and others added 3 commits June 20, 2023 14:53
EsupPortail#3.3.0

- Import external video from url, youtube, peertube and BigBlueButton
- Change xapi actor to deal with Moodle
- Update template to BS5.3 and improve compliance for W3C
- Use redis to cache session and improve logging
- refactor of encoding/transcripting to move it in separate application
- Fixbug on categories, recorder, user liste, tags cloud
@pampletousse pampletousse self-assigned this Aug 28, 2023
@pampletousse pampletousse changed the title Pampletousse/fix accomodation years date delete [DONE] Pampletousse/fix accomodation years date delete Aug 28, 2023
@pampletousse pampletousse changed the title [DONE] Pampletousse/fix accomodation years date delete [WIP] Pampletousse/fix accomodation years date delete Aug 28, 2023
@pampletousse
Copy link
Collaborator Author

Les tests ne passent plus j'essaie de les réparer mais en ce qui concerne le fonctionnel à priori c'est bon

@pampletousse pampletousse force-pushed the pampletousse/fix-accomodation_years_date_delete branch from aba9dab to 4314c51 Compare September 5, 2023 07:29
pod/video/models.py Outdated Show resolved Hide resolved
@ptitloup
Copy link
Contributor

Au fait, tous les test unitaires passent maintenant.

Copy link
Collaborator

@AymericJak AymericJak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok pour moi.

Ne reste plus qu'à effectuer quelques changements demandés précédemment :)

@ptitloup
Copy link
Contributor

Est-ce possible de passer cette PR en DONE ? Merci

@pampletousse pampletousse changed the title [WIP] Pampletousse/fix accomodation years date delete [DONE] Pampletousse/fix accomodation years date delete Sep 18, 2023
Copy link
Contributor

@ptitloup ptitloup left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok pour moi

Copy link
Collaborator

@AymericJak AymericJak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est bon pour moi aussi

@ptitloup ptitloup merged commit 2bceebc into EsupPortail:develop Sep 18, 2023
5 checks passed
vsabatie pushed a commit to vsabatie/Pod that referenced this pull request Nov 22, 2023
* Fix date_delete setup with accommodation years (not correctly setup cause to multiples affiliations per users)

* Add verification for user affiliation to pass tests

* Fix bad parenthesis

* Add verification on affiliation_years result to pass test

* Update test with new working new_year calculation

* Fix tests and save video obsolescence to handle affiliation : array of string OR simple string

* Try fixing Flake8 complexity

* Add pydoc for new function get_date_delete_for_affiliation

* Decompose function to test affiliation, add pydoc

---------

Co-authored-by: Ptitloup <[email protected]>
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.

4 participants