Skip to content

Conversation

@MaudeLaflamme
Copy link
Collaborator

@MaudeLaflamme MaudeLaflamme commented Jan 18, 2023

Voir issue #117

Fix du premier bug:
bao-file-preview_fixed

Fix du deuxième bug:
no-guiding-text_fixed

…st is provided. close VilledeMontreal#117.

Signed-off-by: Maude LAFLAMME <maude.laflamme@montreal.ca>
@MaudeLaflamme MaudeLaflamme added the bug 🔥 Something isn't working label Jan 18, 2023
@MaudeLaflamme MaudeLaflamme requested a review from a team as a code owner January 18, 2023 22:11
@MaudeLaflamme MaudeLaflamme self-assigned this Jan 18, 2023
@gregpetit
Copy link
Contributor

@MaudeLaflamme Peux-tu créer un build dans chromatic pour que je puisse regarder ça?

@MaudeLaflamme
Copy link
Collaborator Author

@gregpetit C'est fait!

@OlivierAlbertini
Copy link
Member

OlivierAlbertini commented Jan 19, 2023

ne pas oublier la PR pour master (Angular 14)

Peut être mettre le close #117 dans la PR pour Angular 14

Copy link
Contributor

@gregpetit gregpetit left a comment

Choose a reason for hiding this comment

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

@MaudeLaflamme J'ai mis mes commentaires dans Chromatic pour essayer. Laisse-moi savoir si c'est correct ou si tu préfères qu'on fasse ça dans GitHub comme d'habitude.

@MaudeLaflamme
Copy link
Collaborator Author

@gregpetit C'est bien les commentaires sur Chromatic! J'y ai répondu

@MaudeLaflamme
Copy link
Collaborator Author

@gregpetit Est-ce qu'il y avait d'autres changements que tu demandais, outre les bugs dans la composante Summary?

@gregpetit
Copy link
Contributor

Non, tout est beau!

@gregpetit gregpetit self-requested a review January 20, 2023 15:12
@MaudeLaflamme MaudeLaflamme merged commit 1c1bf07 into VilledeMontreal:master-13 Jan 20, 2023
@MaudeLaflamme MaudeLaflamme deleted the feature/fix-file-display-bugs branch January 20, 2023 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug 🔥 Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bugs d'affichage des composantes bao-file-input et bao-file-preview

4 participants