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

feat: video gallery option for caption #747

Merged
merged 10 commits into from
Sep 4, 2024
Merged

feat: video gallery option for caption #747

merged 10 commits into from
Sep 4, 2024

Conversation

Wagner3UB
Copy link

Cambiata label per la didascalia video
Aggiunto checkbox per mostrare o meno la didascalia

https://redturtle.tpondemand.com/entity/57952-gallery-video

@Wagner3UB Wagner3UB requested a review from SaraBianchi August 23, 2024 14:43
@Wagner3UB Wagner3UB self-assigned this Aug 23, 2024
@SaraBianchi
Copy link

Cambiata label per la didascalia video Aggiunto checkbox per mostrare o meno la didascalia

https://redturtle.tpondemand.com/entity/57952-gallery-video

Come hai scritto nel commento per me questa era una feature nuova, andrei a modificare il tag e a scriverlo anche tra le novità del release.log il resto è a posto

@Wagner3UB Wagner3UB changed the title fix: video gallery feat: video gallery option for caption Aug 27, 2024
@Wagner3UB
Copy link
Author

@SaraBianchi Fatto

@Wagner3UB
Copy link
Author

@giuliaghisini Anche questa è da approvare

@pnicolli pnicolli added the v2 label Sep 3, 2024
@Wagner3UB
Copy link
Author

@giuliaghisini il corretto, allora, è lasciare la condizione solo a chi ha data.allowExternals, giusto?

@giuliaghisini
Copy link

@giuliaghisini il corretto, allora, è lasciare la condizione solo a chi ha data.allowExternals, giusto?

no, quello non centra nulla con la caption del video

@Wagner3UB
Copy link
Author

@giuliaghisini dovrebbe essere ok adesso, riesci a fare una ulteriore verifica, per favore?

@giuliaghisini giuliaghisini merged commit b2ceb27 into v2 Sep 4, 2024
4 checks passed
@mamico mamico deleted the video-gallery branch November 7, 2024 07:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants