forked from memeLab/Jandig
-
Notifications
You must be signed in to change notification settings - Fork 1
Sprint 4
João Pedro José edited this page Mar 24, 2022
·
21 revisions
11/03/2022 até 24/03/2022
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 | 261 |
Pull Request | |
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 |
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 |