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

[Tech] Retirer la colonne obsolète 'hidden_at' des dossiers (part 1) #10710

Merged
merged 2 commits into from
Sep 2, 2024

Conversation

lisa-durand
Copy link
Contributor

@lisa-durand lisa-durand commented Aug 22, 2024

Suite à la PR #10488
Nettoyage du code pour la colonne hidden_at qui est obsolète.
Une fois déployée il y aura la 2e PR pour supprimer la colonne avec une migration.

Copy link

codecov bot commented Aug 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.58%. Comparing base (9ed2de9) to head (8722a10).
Report is 67 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #10710      +/-   ##
==========================================
+ Coverage   80.28%   84.58%   +4.30%     
==========================================
  Files        1253     1118     -135     
  Lines       26623    24769    -1854     
  Branches     4775     4608     -167     
==========================================
- Hits        21373    20950     -423     
+ Misses       5250     3819    -1431     

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

@lisa-durand lisa-durand force-pushed the remove-unused-hidden-at-for-dossiers-ldu branch from 1dd6acc to b586f23 Compare August 22, 2024 15:37
@lisa-durand lisa-durand changed the title [Tech] Retirer la colonne obsolète 'hidden_at' des dossiers [Tech] Retirer la colonne obsolète 'hidden_at' des dossiers (part 1) Aug 22, 2024
@lisa-durand lisa-durand marked this pull request as ready for review August 22, 2024 16:02
@tchak tchak added this pull request to the merge queue Aug 22, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 22, 2024
@lisa-durand lisa-durand added this pull request to the merge queue Sep 2, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 2, 2024
@lisa-durand lisa-durand added this pull request to the merge queue Sep 2, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 2, 2024
@colinux colinux added this pull request to the merge queue Sep 2, 2024
Merged via the queue into main with commit cb90273 Sep 2, 2024
18 checks passed
@colinux colinux deleted the remove-unused-hidden-at-for-dossiers-ldu branch September 2, 2024 12:49
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.

3 participants