Skip to content
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

Merged
merged 20 commits into from
Jan 23, 2024
Merged

Feature/tarefas #16

merged 20 commits into from
Jan 23, 2024

Conversation

Luckvc
Copy link
Contributor

@Luckvc Luckvc commented Jan 19, 2024

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
    3wXRjTftaf

  • Tela de exibição de tarefa
    image

  • Tela de listagem de tarefas
    g3JZRuSJ8t

  • Tela de edição de tarefas
    image

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.

Luckvc and others added 11 commits January 18, 2024 09:29
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]>
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]>
Copy link

@OtavioLins OtavioLins left a 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

spec/models/user_role_spec.rb Outdated Show resolved Hide resolved
spec/system/tasks/user_register_task_spec.rb Outdated Show resolved Hide resolved
spec/system/tasks/user_view_task_index_spec.rb Outdated Show resolved Hide resolved
@valerialrc
Copy link
Contributor

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.

@OtavioLins
Copy link

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!

Copy link

@OtavioLins OtavioLins left a 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? :)

Luckvc and others added 2 commits January 22, 2024 17:28
Co-authored-by: Valéria Carneiro <[email protected]>
@valerialrc valerialrc mentioned this pull request Jan 22, 2024
@Luckvc
Copy link
Contributor Author

Luckvc commented Jan 22, 2024

@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.

Luckvc and others added 2 commits January 22, 2024 18:51
@Luckvc Luckvc requested a review from OtavioLins January 22, 2024 22:12
Copy link

@OtavioLins OtavioLins left a 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

Luckvc and others added 2 commits January 23, 2024 09:11
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]>
@Luckvc
Copy link
Contributor Author

Luckvc commented Jan 23, 2024

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

Arrumado
image

Adicionamos um bugfix no seeds que resolve a issue #29

@Luckvc Luckvc merged commit 585f117 into main Jan 23, 2024
2 checks passed
@Luckvc Luckvc deleted the feature/tarefas branch January 23, 2024 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants