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

Add sports pass fee #427

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add sports pass fee #427

wants to merge 2 commits into from

Conversation

mmarchois
Copy link
Collaborator

@mmarchois mmarchois commented Mar 14, 2024

Dans le cadre du projet abonnement sportif (voir le détail ici : https://wiki.fairness.coop/fr/references/d%C3%A9cisions/propositions/abonnement-sportif) cette PR permet de rajouter un champ sportsPassFee dans les informations administratives de l'utilisateur.
Ce champ se retrouvera exporté dans les éléments de paies.

👀

image
image

@mmarchois mmarchois force-pushed the feat/add-sports-pass-fee branch from 212dbf4 to 3ecf653 Compare March 14, 2024 15:16
@mmarchois mmarchois self-assigned this Mar 14, 2024
@mmarchois mmarchois force-pushed the feat/add-sports-pass-fee branch 4 times, most recently from 8ec2fab to 70ed697 Compare March 14, 2024 15:49
@mmarchois mmarchois force-pushed the feat/add-sports-pass-fee branch from 70ed697 to 8bace28 Compare March 14, 2024 15:53
@mmarchois mmarchois marked this pull request as ready for review March 14, 2024 15:55
Copy link

codecov bot commented Mar 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.39%. Comparing base (42b3d96) to head (7c4a3ca).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #427      +/-   ##
==========================================
+ Coverage   90.35%   90.39%   +0.04%     
==========================================
  Files         189      189              
  Lines        2219     2229      +10     
  Branches      159      161       +2     
==========================================
+ Hits         2005     2015      +10     
  Misses        205      205              
  Partials        9        9              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Volubyl
Copy link
Contributor

Volubyl commented Mar 25, 2024

👋 ne pas merger cette PR tant que la décision n'a pas été prise concernant l'occtroi de cette prime. La prise de décision aura forum ouvert d'avril

@florimondmanca
Copy link
Contributor

Je retire le label car la mise en place de l'indemnité sport a été décidée et la DUE établie 👍

@mmarchois
Copy link
Collaborator Author

@florimondmanca @Volubyl actuellement, il s'agit d'un export mensuel comme le forfait mobilité durable, et donc envoyé chaque mois à la compta. Or, nous avions dit que le versement de l'indemnité se ferait en septembre. Comment vous voyez les choses dans permacoop ?

@florimondmanca
Copy link
Contributor

florimondmanca commented Jul 2, 2024

@mmarchois Je pense qu'il faut que le champ "Abonnement sportif" soit exprimé par un montant annuel (par ex 480€), le mentionner dans le nom du champ : "Abonnement sportif (montant annuel)"

Pour la transmission à la compta, selon moi on devrait exclure "Abonnement sportif" du tableau des éléments de paie (qui est un tableau mensuel), et avoir un autre tableau à télécharger pour le mois où on fait le versement (septembre mais je pense qu'on devrait pas mettre cette connaissance dans Permacoop)

Pour rester simples je pense qu'on pourrait ajouter une page "Abonnement sportif" dans FairRH qui affiche juste un tableau avec le Nom prénom et le montant annuel, et on met un bouton "Télécharger" ainsi qu'un lien vers le Wiki Abonnement sportif (à créer s'il n'existe pas encore ?)

La page "Eléments de paie" pourrait être transformée mais actuellement elle est très orientée "mois" donc faudrait complètement la refondre pour qu'on puisse y afficher différentes "vues" ...

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

Successfully merging this pull request may close these issues.

4 participants