-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature/tarefas #16
Feature/tarefas #16
Conversation
Co-authored-by: Valéria Carneiro <[email protected]>
Co-authored-by: Valéria Carneiro <[email protected]> Co-authored-by: André Pereira <[email protected]>
Co-authored-by: Valéria Carneiro <[email protected]>
Co-authored-by: Valéria Carneiro <[email protected]>
Co-authored-by: Valéria Carneiro <[email protected]>
Co-authored-by: Valéria Carneiro <[email protected]>
Co-authored-by: Valéria Carneiro <[email protected]>
Co-authored-by: Valéria Carneiro <[email protected]>
Feito o merge para evitar debitos com os critérios de aceite Co-authored-by: Valéria Carneiro <[email protected]>
Co-authored-by: Valéria Carneiro <[email protected]>
Co-authored-by: Valéria Carneiro <[email protected]>
Co-authored-by: Valéria Carneiro <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fiquei confuso com uma coisa: só o dono de um projeto deve poder fazer tarefas? Se sim, ficou faltando autorizações e testes para isso, onde pessoas que não são o autor do projeto tentam criar essas tasks. Seria bom ter uns testes de request também
A parte de autorizações nós deixamos pendente propositalmente porque precisamos que o papel do usuário dentro do projeto esteja bem definido. Conforme a documentação, todos os usuários que colaboram com um projeto podem criar tarefas dentro dele, no caso, só seria limitado o acesso para usuários que não façam parte do projeto. |
Beleza então! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Não esqueçam de mexer nos nomes dos testes antes de fazer o merge, ok? :)
Co-authored-by: Valéria Carneiro <[email protected]>
Co-authored-by: Valéria Carneiro <[email protected]>
@OtavioLins incorporamos o outro PR #19 nesse, essa PR tem a aprovação, mas vou pedir que revise a parte que adicionamos e nos avise quando for aprovado para fazer o merge. |
Link havia desaparecido com o merge. Co-authored-by: Valéria Carneiro <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Passei por tudo e acredito que as novas atualizações também estejam ok. A única questão é a de lembrar de voltar para tratar a autorização no futuro, pois daí bastante coisa tem que ser adicionada, principalmente em relação aos testes de tasks, beleza?
Além disso, vi que nos prints vocês não estilizaram alguns botões do jeito que o resto estão estilizados. Precisa arrumá-los para a aplicação ficar coesa
Co-authored-by: Valéria Carneiro <[email protected]>
seeds estavam criando um perfil duplo para cada usuário. Co-authored-by: Paulo Henrique Meneses <[email protected]>
Adicionamos um bugfix no seeds que resolve a issue #29 |
Alcançado com este pull request
Esse PR resolve a issue #6
Esse PR resolve a issue #17
Esse PR resolve a issue #20
Nesse PR fizemos a criação, exibição e listagem das tarefas de um projeto sendo apenas o campo de título obrigatório. Autor e projeto são associados automaticamente.
Os campos de descrição, prazo e responsável são opcionais. Com validação no campo de prazo para aceitar apenas datas >= hoje.
Adicionamos a edição de tarefas.
Adicionamos o sistema de status da tarefa, com Não iniciada, Em andamento, Finalizada, Cancelada e Expirada.
Tela de criação de tarefa
Tela de exibição de tarefa
Tela de listagem de tarefas
Tela de edição de tarefas
Débitos:
Autorização de acesso às páginas de tarefas ficou a ser feito após o sistema de adição de colaboradores, foi criado um item no backlog para ser realizado futuramente.
Adição de documentos à tarefas ficou a ser feito após o sistema de documentos do projeto, para que os mesmos possam ser adicionados a uma pasta geral.
O sistema automático de mudança de status quando uma tarefa expira entrou como outro item no backlog para ser feito com um job.