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

refactor(dossier): cleanup user controller #11067

Merged
merged 6 commits into from
Nov 26, 2024

Conversation

tchak
Copy link
Member

@tchak tchak commented Nov 21, 2024

Ce nettoyage est fait pour préparer le passage vers l'édition par streams :

  • last_champ_updated_at est touché seulement sur les brouillons
  • On passe explicitement @dossier_for_editing aux views. Ça permet de driver l'usage depuis le controller sans logique dans les views. Cette variable disparaîtra après le passage complet aux streams
  • Supprimer du code mort

@tchak tchak force-pushed the refactor-dossier-controller branch from 5389a0b to e1d6da8 Compare November 21, 2024 15:10
Copy link

codecov bot commented Nov 21, 2024

Codecov Report

Attention: Patch coverage is 96.10390% with 3 lines in your changes missing coverage. Please review.

Project coverage is 84.38%. Comparing base (7932085) to head (1f5dd3b).
Report is 7 commits behind head on main.

Files with missing lines Patch % Lines
app/controllers/users/dossiers_controller.rb 92.59% 2 Missing ⚠️
app/models/concerns/dossier_state_concern.rb 92.30% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #11067      +/-   ##
==========================================
+ Coverage   84.35%   84.38%   +0.03%     
==========================================
  Files        1176     1176              
  Lines       25944    25950       +6     
  Branches     4898     4903       +5     
==========================================
+ Hits        21886    21899      +13     
+ Misses       4058     4051       -7     

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

@tchak tchak force-pushed the refactor-dossier-controller branch from b5c075e to 1f5dd3b Compare November 26, 2024 13:30
@tchak tchak enabled auto-merge November 26, 2024 13:34
@tchak tchak added this pull request to the merge queue Nov 26, 2024
Merged via the queue into demarches-simplifiees:main with commit 755a526 Nov 26, 2024
18 checks passed
@tchak tchak deleted the refactor-dossier-controller branch November 26, 2024 13:47
Copy link

sentry-io bot commented Nov 29, 2024

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ ActionView::Template::Error: virus_scanner_error? delegated to blob, but blob is nil (ActionView::Template::Error) Champs::PieceJustificativeController#update View Issue
  • ‼️ **ActionView::Template::Error: undefined method include?' for false (ActionView::Template::Error)** Users::DossiersController#modifier` View Issue
  • ‼️ **ActionView::Template::Error: undefined method push' for nil (ActionView::Template::Error)** Administrateurs::ProceduresController#apercu` View Issue
  • ‼️ **ActionView::Template::Error: undefined method include?' for false (ActionView::Template::Error)** Users::DossiersController#modifier` View Issue

Did you find this useful? React with a 👍 or 👎

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