ETQ instructeur, j'ai besoin d'un accès très facile à mondernier export #9855
Conversation
There was a problem hiding this comment.
Pour le bug sur les exports en échec: je pense que ça vient du fait que le poll appelle download_export qui fait un .find_or_create_fresh_export, qui finit par un create car le dernier export a été en échec. Ce qui veut dire qu'en l'état, on créé un nouvel export dès le premier poll, et le last_export renvoyé est donc en pending
| export = Export.where(instructeur_id: current_instructeur.id, statut: statut).last | ||
| if export.present? | ||
| return nil if export.updated_at < 1.hour.ago | ||
| end | ||
| export |
There was a problem hiding this comment.
petite simplification :
| export = Export.where(instructeur_id: current_instructeur.id, statut: statut).last | |
| if export.present? | |
| return nil if export.updated_at < 1.hour.ago | |
| end | |
| export | |
| Export.where(instructeur_id: current_instructeur.id, statut: statut, updated_at: 1.hour.ago..).last |
lisa-durand
force-pushed
the
display-last-export-in-same-page-ldu
branch
2 times, most recently
from
9d10a4a to
d948009
Compare
| @@ -0,0 +1,11 @@ | |||
| - if export.present? | |||
| %div{ data: export.pending? ? { controller: "turbo-poll", turbo_poll_url_value: polling_last_export_instructeur_procedure_path(export_format: export.format, statut: @statut), turbo_poll_interval_value: 5_000, turbo_poll_max_checks_value: 6 } : {} } | |||
There was a problem hiding this comment.
je l'avais pas vue avant, mais dans ce partial utilise une variable d'instance @statut. Comme pour l'export ce serait mieux de la passer en variable locale
| @@ -6,3 +6,6 @@ | |||
| - else | |||
| = turbo_stream.update_all '.dossiers-export' do | |||
| = render Dossiers::ExportDropdownComponent.new(procedure: @procedure, statut: @statut, count: @dossiers_count, export_url: method(:download_export_instructeur_procedure_path)) | |||
|
|
|||
| = turbo_stream.update "last-export-alert" do | |||
| = render partial: "last_export_alert", locals: {export: @last_export} | |||
There was a problem hiding this comment.
vu que tu vas modifier les locals, il manque les espaces autour des {}(d'ailleurs en principe haml-lint est censé les voir ?)
lisa-durand
force-pushed
the
display-last-export-in-same-page-ldu
branch
from
d948009 to
3e79278
Compare
lisa-durand
force-pushed
the
display-last-export-in-same-page-ldu
branch
from
3e79278 to
e6a4459
Compare
github-merge-queue
bot
removed this pull request from the merge queue due to failed status checks
closes #9736