This repository has been archived by the owner on Jul 22, 2022. It is now read-only.
[Roadmap] Formalisation de l'affichage des tables des modules HR et responsable #178
+833
−948
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
1 task
|
Différence avec la PR #179 ? |
|
Il y a une différence fondamentale : cette PR s'occupe de HR et du responsable et pas l'autre 😆 Ces PR sont massives et si je peux le découper, je le fais, de tel sorte que la review et le test soit plus aisé et isolé. Dans le cas contraire, certaines choses non souhaitées peuvent glisser entre les mailles. |
| @@ -1054,7 +1098,7 @@ public static function affichage($user_login, $year_affichage, $year_calendrier | |||
| $return .= '<h3>' . _('resp_traite_user_etat_demandes_2_valid') . '</h3>'; | |||
|
|
|||
| //affiche l'état des demande en attente de 2ieme valid du user (avec le formulaire pour le responsable) | |||
| $return .= affiche_etat_demande_2_valid_user_for_resp($user_login); | |||
|
vu |
| 'table-condensed', | ||
| 'table-responsive', | ||
| ]); | ||
| $childTable .= '<thead>'; |
|
Testé. J'ai remarqué, avec la double validation, que le RH ne voyait pas les demandes ayant été validé au 1er niveau et en attente de validation du grand responsable, je ne sais pas si c'est le cas sur la stable, je ferai un essai dans la journée... |
|
C'est confirmé, le problème est présent sur Quinnis aussi... |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Statut :
Cf. #103
Dans l'attente de pouvoir continuer à bosser sur la gestion des heures, j'ai poursuivi mon travail sur la mise en template, les modules HR et responsable pour cette PR ci. Comme la PR précédente, il s'agit d'utiliser l'objet
Tablepour construire les tables des modules et ainsi d'avoir un « moule » commun.L'objectif sous-jacent est triple :
J'en ai profité pour passer un petit coup de balai sur des fonctions qui n'étaient pas utilisées dû à une migration antérieure de code. Et corrigé un bug, si tu veux y jeter un œil...