Skip to content

Hotfix new prez#7

Merged
Liv44 merged 2 commits intodevelopfrom
hotfix-new-prez
Apr 8, 2023
Merged

Hotfix new prez#7
Liv44 merged 2 commits intodevelopfrom
hotfix-new-prez

Conversation

@Liv44
Copy link
Collaborator

@Liv44 Liv44 commented Apr 8, 2023

  • Input duration en type number
  • Les participants deviennent un array de string
  • Problème de nom de fichier géré pour windows et mac
  • Problème d'extension du fichier codePrez lors de la sauvegarde pour Windows géré

@Liv44 Liv44 requested a review from matheoleger April 8, 2023 23:18
Copy link
Owner

@matheoleger matheoleger left a comment

Choose a reason for hiding this comment

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

Fix my comment and we're good !


const createCodePrez = () => {
const arrayAuthors = authors.split(',').map((author) => {
return author.trimEnd().trimStart();
Copy link
Owner

Choose a reason for hiding this comment

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

Suggested change
return author.trimEnd().trimStart();
return author.trim();

@Liv44 Liv44 force-pushed the hotfix-new-prez branch from 16a18b8 to 266d0a0 Compare April 8, 2023 23:26
@Liv44 Liv44 requested a review from matheoleger April 8, 2023 23:27
@Liv44 Liv44 merged commit 89476cb into develop Apr 8, 2023
@Liv44 Liv44 deleted the hotfix-new-prez branch April 8, 2023 23:28
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