Skip to content

Update include_video.feature#394

Draft
NourhaneBoudaoud wants to merge 1 commit intomainfrom
fix-365
Draft

Update include_video.feature#394
NourhaneBoudaoud wants to merge 1 commit intomainfrom
fix-365

Conversation

@NourhaneBoudaoud
Copy link
Copy Markdown

We, [FILL WITH FULL NAMES], hereby grant to Hyperglosae maintainers the right to publish our contribution under the terms of any licenses the Free Software Foundation classifies as Free Software Licenses.

@NourhaneBoudaoud NourhaneBoudaoud self-assigned this Mar 24, 2026
@NourhaneBoudaoud NourhaneBoudaoud added the Good first issue Issue that is easy to test and fix for new team members or external contributors label Mar 24, 2026
@NourhaneBoudaoud NourhaneBoudaoud linked an issue Mar 24, 2026 that may be closed by this pull request
6 tasks
Copy link
Copy Markdown
Member

@benel benel left a comment

Choose a reason for hiding this comment

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

Thank you for your pull request @Hypertopic/tutos.

To improve other scenarios to come, please read commented lines.

Please note also that commit titles should be very strictly named (check Moodle for the rules to apply).

For this particular enhancement, there is a far simpler way to think about this improvement: see what I wrote on the issue discussion (#365). You will see that the scenario (and tests and implementation) will be far easier to update.
You only have to change one line in this scenario.




Scénario: via le bouton d'insertion de l'interface
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please don't mention user interface details in scenarios.


Soit un document dont je suis l'auteur affiché comme glose
Et une session active avec mon compte
Quand je clique sur l'icône "Add a video"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please don't mention user interface details in scenarios.

Soit un document dont je suis l'auteur affiché comme glose
Et une session active avec mon compte
Quand je clique sur l'icône "Add a video"
Et que je saisis le lien "https://www.youtube.com/watch?v=JRXkAhMYKEc&ab_channel=ViniciusHenrique" dans la boîte de dialogue
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

When (Quand) must be a single step. It corresponds to the feature. Therefore it should be named in a way very similar to the feature name.

Et une session active avec mon compte
Quand je clique sur l'icône "Add a video"
Et que je saisis le lien "https://www.youtube.com/watch?v=JRXkAhMYKEc&ab_channel=ViniciusHenrique" dans la boîte de dialogue
Et que je valide l'insertion
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

When (Quand) must be a single step. It corresponds to the feature. Therefore it should be named in a way very similar to the feature name.

Et que je saisis le lien "https://www.youtube.com/watch?v=JRXkAhMYKEc&ab_channel=ViniciusHenrique" dans la boîte de dialogue
Et que je valide l'insertion
Alors le document comporte la vidéo "https://www.youtube.com/embed/JRXkAhMYKEc"
Et le texte brut de la glose contient "![link](https://www.youtube.com/watch?v=JRXkAhMYKEc)"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Not necessarily.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Good first issue Issue that is easy to test and fix for new team members or external contributors

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Simplified Media Insertion (Video/Photo)

3 participants