Skip to content

Ajoute bouton pour valider les work entries#1

Open
Hugo-Trentesaux wants to merge 18 commits into18.0-add_hr_work_entryfrom
18.0-hugo-owl
Open

Ajoute bouton pour valider les work entries#1
Hugo-Trentesaux wants to merge 18 commits into18.0-add_hr_work_entryfrom
18.0-hugo-owl

Conversation

@Hugo-Trentesaux
Copy link
Member

Ajoute un bouton sur la vue calendrier des work entries :

  • en vue mois
  • en vue semaine
  • pas dans les autres vues (jour année)

Le bouton est désactivé quand aucune entrée de la période n'est en brouillon.

Quand au moins une entrée est en brouillon, cliquer sur le bouton valide toutes les entrées en brouillon de la période en cours. Une notification est affichée avec le nombre d'entrées validées en cas de succès. Une notification d'erreur est affichée en cas d'erreur.


J'ai mis un peu trop de temps pour en arriver là, mais ça m'a permis de mieux comprendre la vue calendrier et owl. Par contre, je ne suis pas satisfait de la méthode de patch. J'aimerais pouvoir patcher une version du calendar controller uniquement utilisée pour les work entries, mais comme c'est le calendar controller générique qui est utilisé, on patche tout et donc le patch sera présent dans n'importe quelle vue calendrier (même si inutilisé).

@remi-filament
Copy link
Member

Merci @Hugo-Trentesaux
Il faudrait que tu corriges pre-commit stp ?
Pour la surcharge JS, en effet c'est pas top, est-ce que c'est pas possible de faire comme ici : https://github.com/odoo/odoo/blob/18.0/addons/hr_work_entry_contract/static/src/views/work_entry_calendar/work_entry_calendar_controller.js
(c'est le JS qui permet d'ajouter le bouton pour Regénérer les work entries) ?

Tu l'as sûrement déjà étudié, mais il me semble que ça fait ce que tu voudrais faire non ?

<?xml version="1.0" encoding="UTF-8" ?>
<templates xml:space="preserve">
<t
t-name="hr_work_entry_validate.calendar.controlButtons"
Copy link
Member

Choose a reason for hiding this comment

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

Il te faut pas garder le t-name ?

Copy link
Member Author

Choose a reason for hiding this comment

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

Je l'ai remis

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