forked from memeLab/Jandig
-
Notifications
You must be signed in to change notification settings - Fork 1
Sprint 4
Bruno Félix edited this page Apr 26, 2022
·
21 revisions
11/03/2022 até 24/03/2022
Esta sprint teve como objetivo dar continuidade às contribuições no projeto e realizar as melhorias mapeadas em sprints anteriores. Esta sprint também teve como primeiro contato as revisões, por parte da comunidade Jandig, de PR's feitos anteriormente na disciplina, desta forma, a implementação das mudanças apontadas nas revisões também foi um objetivo.
Nome | User no GitHub |
---|---|
Bruno Félix | @Bruno-Felix |
Danilo Domingo | @Danilow200 |
Edvan Gomes | @Eddie-gomes |
Gabrielle Ribeiro | @Gabrielle-Ribeiro |
Gustavo Afonso | @GustavoAPS |
Gustavo Duarte | @gustavoduartemoreira |
Hugo Sobral | @KiSobral |
João Pedro José | @sudjoao |
Lucas Lima | @mibasFerraz |
Nicalo Ribeiro | @nicaloribeiro |
Sofia Patrocínio | @sofiapatrocinio |
Victor Amaral | @VictorAmaralC |
Atividade | Descrição |
---|---|
Issue | No login error message when there's a space in the field |
Pull Request | 428 - No login error message when there's a space in the field |
O que foi feito | O problema foi corrigido conforme especificado na issue. |
Problemas | Apesar do problema ter sido resolvido conforme especificado na issue, essa solução não segue o padrão de outras partes do código, onde os espaços em branco são somente ignorados e nenhuma mensagem de erro é disparada. Além disso, no mesmo código em outros trechos encontramos algumas inconsistências, comentários de FIX-ME e códigos que são excelentes candidatos para refatoração. |
Solução | Estamos discutindo com os mantenedores as possibilidades, pois podemos seguir o padrão do resto do código de ignorar os espaços em branco. Entretanto, esse comportamento de ignorar os espaços em branco não é o mais adequado para se lidar com a situação em que o usuário digita um espaço em branco no email/username no cadastro e no login. Se formos optar por lidar com a situação da forma mais adequada, isto é, disparando um erro quando houver um espaço em branco, todo o código deverá ser refatorado e para isso um nova issue deverá ser aberta. |
Atividade | Descrição |
---|---|
Issue | Change the sender name of the password recovery email to Jandig |
Pull Request | 293 - Change the sender name of the password recovery email to Jandig |
O que foi feito | O problema foi corrigido conforme especificado na issue. Utilizamos um email diferente do Jandig para testar se as modificações funcionam. |
Problemas | Foi encontrado um problema no sistema de recuperação de email, onde no próprio site do Jandig, quando se tenta recuperar a senha, no momento em o email com a recuperação de senha deve ser enviado para o usuário, o código quebra. |
Solução | Foi aberta uma issue #450 para reportar o problema de recuperação de email. Entretanto, essa issue só pode ser corrigida pelos mantenedores, já que o problema provavelmente está nas variáveis de ambiente no ambiente de deploy ou nas permissões para o acesso da SMTP ao servidor do email. |
Atividade | Descrição |
---|---|
Issue | As a developer, I want to find easily an updated code of conduct, so I can read and understand it |
Pull Request | 290 - As a developer, I want to find easily an updated code of conduct, so I can read and understand it |
O que foi feito | O código de conduta foi atualizado para a versão 2.0. |
Problemas | - |
Solução | - |
Atividade | Descrição |
---|---|
Issue | Problem to Run Jandig ARte |
Pull Request | 436 - Problem to Run Jandig ARte |
O que foi feito | Foram mapeados os problemas que impediam o grupo de rodar o projeto e com isso os requirements foram atualizados. |
Problemas | - |
Solução | - |
Atividade | Descrição |
---|---|
Issue | update translation .po |
Pull Request | 241 - Update translation |
O que foi feito | Ajuste na tradução de mensagens de erro em português |
Problemas | - |
Solução | Mudança na tradução de mensagens de erro |
Atividade | Descrição |
---|---|
Issue | As an user, I want to start writting a name of an exhibition and see it, so I don't need to find it in a long drop-down menu. |
Pull Request | 261 - Improve dropdown menu exhibition |
O que foi feito | Criação de um novo componente, através das classes do django, que possui o comportamento de listagem e pesquisa de exibições.. |
Problemas | Dificuldade em entender como estava sendo criado o componente atual, dificuldade de documentação do django sobre a criação de elementos do tipo datalist. |
Solução | Criação de uma classe que retorna um input com o datalist e uma classe que retorna um form com esse datalist |
Atividade | Descrição |
---|---|
Issue | Revisar o PR 367 - Fix frontend error message for duplicated exhibit name and slugs #383 |
Pull Request | 367 - Fix frontend error message for duplicated exhibit name and slugs |
O que foi feito | Correção do texto presente nas mensagens para o usuário e tradução. Verificação de nomes duplicados e tradução dos textos presentes nas mensagens |
Problemas | recuperação do código dos outros repositórios (da disciplina em semestres anteriores) e entendimento do código realizado pelos estudantes do semestre anterior para realização da alteração solicitada |
Solução | Alteração dos textos e traduções solicitadas e correção das mensagens de erro para nome e slugs. E alteração da exhibit |
Atividade | Descrição |
---|---|
Issue | 289 - As an artist, I want to receive a warning when I try to upload the wrong file format, so I can publish the right one |
Pull Request | -- |
O que foi feito | -- |
Problemas | O programa não avisa quando se tenta fazer upload de um tipo arquivo não aceito. |
Solução | -- |
Pull Request | Descrição | Estado |
---|---|---|
376 - Create profile page | Remoção de código inutilizado | Mergeado |
Pull Request | Descrição | Estado |
---|---|---|
445 - Fix prospector errors | Solução de violações de código que passaram em branco e criação de documentação sobre o processo e a ferramenta | Mergeado |
Data | Versão | Atividade | Responsável |
---|---|---|---|
23/03/2022 | 0.1 | Criação e Desenvolvimento da Wiki Sprint 4 | Gabrielle Ribeiro |
23/03/2022 | 0.2 | Adição de informações sobre a Issue 261 | João Pedro Guedes |
24/03/2022 | 0.3 | Adição do link do PR da Issue 261 e informações da Issue 241 | Sofia Patrocínio |
24/03/2022 | 0.4 | Adição do objetivo da sprint e da sessão de evoluções feitas em PR's anteriores | Hugo Sobral |
24/03/2022 | 0.5 | Adição de informações e link PR's 456 | Gustavo Duarte Moreira |