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.

Sign up for GitHub

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

Jump to bottom

Statistiques: Indicateurs à suivre accessible aux membres de l'équipe des emplois [GEN-2190] #5007

Merged
merged 1 commit into from
Nov 12, 2024
Merged

Statistiques: Indicateurs à suivre accessible aux membres de l'équipe des emplois [GEN-2190] #5007

merged 1 commit into from
Nov 12, 2024

Conversation

calummackervoy
Copy link
Contributor

🤔 Pourquoi ?

Ce fonctionnalité permet les membres de l'équipe des emplois d'accéder aux stats indicateurs clés de la service

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?

💻 Captures d'écran

Screenshot 2024-10-28 at 15 56 38

@calummackervoy calummackervoy added 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC ajouté Ajouté dans le changelog. labels Oct 28, 2024
@calummackervoy calummackervoy self-assigned this Oct 28, 2024
@calummackervoy calummackervoy changed the title Statistiques: Indicateurs à suivre accessible aux membres de l'équipe des emplois Statistiques: Indicateurs à suivre accessible aux membres de l'équipe des emplois [GEN-2190] Oct 28, 2024
@notion-workspace Notion Workspace
Copy link

[Tableau de bord] Intégrer le TB 438 “Indicateurs à suivre - Emplois” à l’espace statistique pour les membres de l’équipe

@github-actions GitHub Actions
Copy link

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

@calummackervoy calummackervoy removed the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Oct 28, 2024
@calummackervoy calummackervoy force-pushed the calum/tb-staff-indicateurs-emplois branch 2 times, most recently from 3d1ceca to f56d56f Compare October 28, 2024 17:37
rsebille
rsebille reviewed Nov 4, 2024
View reviewed changes
itou/www/stats/urls.py Outdated Show resolved Hide resolved
itou/templates/dashboard/includes/staff_stats_card.html Outdated Show resolved Hide resolved
tests/www/stats/test_utils.py Outdated Show resolved Hide resolved
@calummackervoy calummackervoy force-pushed the calum/tb-staff-indicateurs-emplois branch from f56d56f to 8383565 Compare November 4, 2024 15:35
@calummackervoy
Staff users can access indicator stats for Les Emplois via stats view
05b85bd 
Add permissions function to solution for consistency

Reinstate files lost during rebase

Refactor TdB URL name to follow convention

Fix missing template usage

Remove low value comment

Rename TdB to include user kind
@calummackervoy calummackervoy force-pushed the calum/tb-staff-indicateurs-emplois branch from 8383565 to 05b85bd Compare November 7, 2024 15:26
rsebille
rsebille approved these changes Nov 7, 2024
View reviewed changes
Copy link
Contributor

@rsebille rsebille left a comment

Choose a reason for hiding this comment

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

🎉, je pense que Marion et Zohra vont beaucoup aimer cette MEP 😀

@calummackervoy calummackervoy added this pull request to the merge queue Nov 12, 2024
Merged via the queue into master with commit 52b2ba5 Nov 12, 2024
9 checks passed
@calummackervoy calummackervoy deleted the calum/tb-staff-indicateurs-emplois branch November 12, 2024 07:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rsebille rsebille rsebille approved these changes

Assignees

@calummackervoy calummackervoy

Labels
ajouté Ajouté dans le changelog.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@calummackervoy @rsebille