Skip to content

[DONE] Ptitloup/fix thumbnail #790

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

Merged
merged 25 commits into from
Apr 3, 2023

Conversation

ptitloup
Copy link
Contributor

  • fix create thumbnail by getting the lower mp4 format

@ptitloup ptitloup mentioned this pull request Mar 29, 2023
@ptitloup ptitloup changed the title [WIP] Ptitloup/fix thumbnail [DONE] Ptitloup/fix thumbnail Mar 29, 2023
Copy link
Contributor

@pampletousse pampletousse left a comment

Choose a reason for hiding this comment

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

Ok pour moi

Copy link
Contributor Author

@ptitloup ptitloup left a comment

Choose a reason for hiding this comment

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

tout me semble ok

@mattbild
Copy link
Collaborator

j'ai une erreur à la suppression d'une vidéo (testé 2 fois)

Cannot add or update a child row: a foreign key constraint fails (`pod`.`video_video`, CONSTRAINT `video_video_thumbnail_id_de05caf2_fk_podfile_customimagemodel_id` FOREIGN KEY (`thumbnail_id`) REFERENCES `podfile_customimagemodel` (`id`))

Je ne sais pas si c'est uniquement chez moi

@ptitloup
Copy link
Contributor Author

Visio ?

@mattbild
Copy link
Collaborator

mattbild commented Mar 30, 2023

c'est bien sur cet env local qu'il y a des problèmes à la supp d'une vidéo.

Toutes les vidéos que je viens de tester ont bien des thumbnails générés et un affiché

Copy link
Contributor Author

@ptitloup ptitloup left a comment

Choose a reason for hiding this comment

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

ok pour moi

@mattbild
Copy link
Collaborator

mattbild commented Mar 31, 2023

A la suppression je n'ai plus les fichiers (sauf le dossier de la vidéo qui est vide)
mais j'ai encore l'entrée en base dans la table video_video
donc la vignette (sans thumbnail) est affichée dans la liste de mes vidéos

@ptitloup
Copy link
Contributor Author

A la suppression je n'ai plus les fichiers (sauf le dossier de la vidéo qui est vide) mais j'ai encore l'entrée en base dans la table video_video donc la vignette (sans thumbnail) est affichée dans la liste de mes vidéos

je constate egalement ce dysfonctionnement

@ptitloup
Copy link
Contributor Author

Je viens normalement de fixer ce dysfonctionnement, tout est bien supprimé maintenant

@mattbild
Copy link
Collaborator

Je viens normalement de fixer ce dysfonctionnement, tout est bien supprimé maintenant

c'est bon pour moi.
La vidéo n'apparait plus et n'existe plus en base.

@mattbild
Copy link
Collaborator

ok :

  • les boutons de suppression de la vidéo n'apparaissent plus tant qu'elle est en cours d'encodage
  • on n'a plus l'erreur concernant le propriétaire de la vidéo quand on la modifie pendant l'encodage

@ptitloup
Copy link
Contributor Author

ok :

* les boutons de suppression de la vidéo n'apparaissent plus tant qu'elle est en cours d'encodage

* on n'a plus l'erreur concernant le propriétaire de la vidéo quand on la modifie pendant l'encodage

Merci ! je dois encore verifier les tests unitaires et ca sera tout bon

@ptitloup ptitloup merged commit edfb546 into EsupPortail:develop Apr 3, 2023
ptitloup added a commit that referenced this pull request Apr 12, 2023
* 🐛 podfile fix pagination display (#787)
* improve clean_video_files and code cleaning (#782)
* Fix wrong url of paginated sub-themes (#789)
* Add "help" command to makefile (#792)
* Ptitloup/fix thumbnail (#790)
* Hide channel desc (#794)
* fix podfile (#798)
* Correct podfile sidebar (#799)
* Ptitloup/fix link playlist (#801)
* Fix video theme (#800)


---------
Co-authored-by: MattBild <34771705+mattbild@users.noreply.github.com>
Co-authored-by: Olivier Bado-Faustin <bado@unice.fr>
Co-authored-by: pampletousse <51699553+pampletousse@users.noreply.github.com>
vsabatie pushed a commit to vsabatie/Pod that referenced this pull request Apr 20, 2023
* 🐛 podfile fix pagination display (EsupPortail#787)
* improve clean_video_files and code cleaning (EsupPortail#782)
* Fix wrong url of paginated sub-themes (EsupPortail#789)
* Add "help" command to makefile (EsupPortail#792)
* Ptitloup/fix thumbnail (EsupPortail#790)
* Hide channel desc (EsupPortail#794)
* fix podfile (EsupPortail#798)
* Correct podfile sidebar (EsupPortail#799)
* Ptitloup/fix link playlist (EsupPortail#801)
* Fix video theme (EsupPortail#800)

---------
Co-authored-by: MattBild <34771705+mattbild@users.noreply.github.com>
Co-authored-by: Olivier Bado-Faustin <bado@unice.fr>
Co-authored-by: pampletousse <51699553+pampletousse@users.noreply.github.com>
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.

4 participants