vlgi / uai-components

Apache License 2.0

2 stars 1 forks source link

Feature/create comments components - [merged] #263

Closed 01oseluiz closed 9 months ago

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 19, 2023, 21:57

Merges feature/create-comments-components -> develop

Gitlab MR Web Template

Descrição

Criar componentes para fazer um chat/comentarios

Tipo de mudança

[ ] Bug fix (Correção de um problema que não quebrava algo grande)
[X] New feature (Mudança que adiciona novas funcionalidades)
[ ] Breaking change (correção ou recurso que faria com que a funcionalidade existente não funcionasse como esperado)
[ ] Refactor (Alteração que melhora algo que já existente)

Em quais navegadores foi testado?

[X] Google Chrome
[ ] Mozilla Firefox
[ ] IE 11 (Windows 10)
[ ] Edge
[ ] Opera
[ ] Opera GX
[ ] IOS Safari
[ ] IOS Chrome
[ ] Android Chrome
[ ] Android Firefox

Testei as seguintes resoluções no navegador:

[ ] 2520 x 1080
[X] 1920 x 1080
[ ] 1400 x 790
[X] 1280 x 720
[ ] 1024 x 1366 (iPad Pro vertical)
[ ] 1366 x 1024 (iPad Pro horizontal)
[ ] 768 x 1024 (iPad vertical)
[ ] 1024 x 768 (iPad horizontal)
[ ] 375 x 812 (iPhone X)
[X] 320 x 568 (iPhone 5/SE)

Checklist:

Qualidade

[X] Segui os padrões de criação da stack em svelte
[X] Eu revisei ou reli o meu próprio código
[X] Segui o formato de criação e nomenclatura de variáveis CSS

Documentação

[X] Documentei onde era mais necessário e difícil de entender no meu código
[X] Documentei as variáveis CSS que meu componente externaliza
[X] Criei bons exemplos no storybook para meu componente

Teste

[ ] As mudanças que esta MR implementa possui testes automatizados
[X] Executei e garanto que os testes automatizados já criados estão todos passando
[X] Executei e garanto que não há erros de lint
[X] Executei e garanto que não há erros de typescript

MR

[X] Eu conferi a checklist de atividades, e expliquei no meu merge request caso algo não tenha sido necessário ou possível

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 20, 2023, 16:59

added 2 commits

bce5cbe5 - ChatInput (fix): adjust chat input
43eb6865 - MessageBox (feat): allow my message in right

Compare with previous version

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 20, 2023, 17:25

added 1 commit

0c3f33cd - ChatInput (fix): remove send button

Compare with previous version

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 21, 2023, 17:18

added 6 commits

0bd28c90 - ChatInput (feat): add exported variables and css variables
36051713 - MessageBox (docs): small adjustments and documentation
5e19c0dd - ChatInput (feat): add EmojiPicker
373895c9 - ChatInput (doc): document variables
a8618d4c - MessageBox (doc): fix name
b780da36 - (fix): fix prettier format

Compare with previous version

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 21, 2023, 17:20

marked the checklist item 1280 x 720 as completed

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 21, 2023, 17:20

marked the checklist item 320 x 568 (iPhone 5/SE) as completed

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 21, 2023, 17:20

marked the checklist item Segui os padrões de criação da stack em svelte as completed

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 21, 2023, 17:20

marked the checklist item Eu revisei ou reli o meu próprio código as completed

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 21, 2023, 17:20

marked the checklist item Segui o formato de criação e nomenclatura de variáveis CSS as completed

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 21, 2023, 17:20

marked the checklist item Documentei onde era mais necessário e difícil de entender no meu código as completed

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 21, 2023, 17:20

marked the checklist item Documentei as variáveis CSS que meu componente externaliza as completed

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 21, 2023, 17:20

marked the checklist item Criei bons exemplos no storybook para meu componente as completed

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 21, 2023, 17:20

marked the checklist item Executei e garanto que os testes automatizados já criados estão todos passando as completed

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 21, 2023, 17:20

marked the checklist item Executei e garanto que não há erros de lint as completed

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 21, 2023, 17:20

marked the checklist item Executei e garanto que não há erros de typescript as completed

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 21, 2023, 17:20

marked the checklist item Eu conferi a checklist de atividades, e expliquei no meu merge request caso algo não tenha as completed

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 21, 2023, 17:21

marked this merge request as ready

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 22, 2023, 14:01

added 2 commits

9031a64b - MessageBox (refactor): update types
2177a69b - ChatInput (doc): update doc

Compare with previous version

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 22, 2023, 14:34

added 1 commit

fcbcdc79 - MessageBox (feat): include buttons in a dropdown

Compare with previous version

01oseluiz commented 1 year ago

In GitLab by @Wander-lan on Sep 22, 2023, 17:13

requested review from @Wander-lan

01oseluiz commented 1 year ago

In GitLab by @Wander-lan on Sep 22, 2023, 19:32

Você chegou a rodar o npm run checl-all? Sei que alguns warnings aí estão fora do escopo dessa MR mas é uma boa prática resolver eles também antes de dar merge. Aí fica ao teu critério resolver ou não.

01oseluiz commented 1 year ago

In GitLab by @Wander-lan on Sep 22, 2023, 19:34

Seria bom acrescentar os eventos no storybook do componente MessageBox. É importante pra quem tiver testando no storybook saber quando esses eventos são trigados.

01oseluiz commented 1 year ago

In GitLab by @Wander-lan on Sep 22, 2023, 19:36

Acho que seria interessante se houvesse um botão de "Enviar" ao lado desses dois. Pois achei meio confuso o fato de não haver um botão ou um atalho pra enviar a mensagem no modo de 'Textarea'

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 22, 2023, 19:47

A documentação dos eventos é automática, o erro era que eles nem estavam sendo envidados.

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 22, 2023, 19:51

Os erros do prettier tudo bem mas os outros são irrelevantes e na verdade corrigir eles quebraria outra coisa, o any nos lugares indicados é necessário

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 22, 2023, 19:55

added 3 commits

6bc2809d - MessageBox (fix): fix send events
c2fcd10c - MessageBox (fix): prettier format
b208fa6d - ChatInput (feat): add send button

Compare with previous version

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 22, 2023, 19:55

resolved all threads

01oseluiz commented 1 year ago

In GitLab by @Wander-lan on Sep 22, 2023, 20:27

approved this merge request

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 25, 2023, 11:56

added 1 commit

aadfb258 - ChatInput (fix): fix id

Compare with previous version

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 25, 2023, 12:58

added 1 commit

5dbbaa5c - ChatInput (feat): alow change icons colors

Compare with previous version

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 25, 2023, 13:08

added 1 commit

be52a71c - ChatInput (fix): fix change buttons colors

Compare with previous version

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 25, 2023, 13:53

added 1 commit

82e9bfa5 - ChatInput (fix): fix dropdown

Compare with previous version

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 25, 2023, 14:24

added 1 commit

63f75b20 - MessageBox (style): adjust font weight

Compare with previous version

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 25, 2023, 14:30

added 1 commit

bda771ca - MessageBox (style): alow change buttons color

Compare with previous version

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 25, 2023, 17:45

added 1 commit

2253fd0c - MessageBox (feat): add image slot

Compare with previous version

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 25, 2023, 18:15

added 1 commit

f62b6ca7 - MessageBox (feat): use atributs in slot

Compare with previous version

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 26, 2023, 14:31

added 1 commit

46fb5ac0 - MessageBox (feat): alow edit message

Compare with previous version

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 27, 2023, 12:17

added 1 commit

c1951f7b - MessageBox (feat): show message updateAt

Compare with previous version

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 27, 2023, 12:39

added 1 commit

663809b7 - MessageBox (style): fix move text on hover

Compare with previous version

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 27, 2023, 12:58

added 1 commit

4bff1d8c - MessageBox (fix): fix click dropdown

Compare with previous version

01oseluiz commented 1 year ago

In GitLab by @victorandre957 on Sep 27, 2023, 13:23

mentioned in commit 4ccdc859a378516b0ed804741a394d6da05110c5