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/lider deleta projeto #21

Merged
merged 8 commits into from
Jan 22, 2024
Merged

Conversation

fredericomozzato
Copy link
Contributor

Alcançado com este PR

Este PR resolve a issue #5 para líder ser capaz de deletar o projeto.

Foi implementado um botão para deletar projeto na página de edição do mesmo. Como a página de edição ainda não havia sido implementada, criamos a view e a action projects#edit.

image

Ao clicar em Deletar uma janela de confirmação é apresentada ao usuário:

image

Caso o usuário confirme deletar o projeto, este não aparece mais na lista de projetos. A deleção redireciona para a view my_projects, que mostra somente os projetos do usuário.

image

Além disso modificamos o teste de requisição de projetos, alterando o nome antigo (spec/requests/projects/user_creates_project_spec.rb) para um nome mais condizente com o padrão do Rails e deixando o teste na página de requests (spec/requests/projects_requests_spec.rb). Adicionamos o bloco geral do RSpec a modificamos os cenários para refletir quais rotas estão sendo testadas.

Débitos

Não foi adicionado o formulário de edição na view edit, nem implementada a action update.

adoniranfranceh and others added 7 commits January 19, 2024 16:45
Co-authored-by: Frederico Mozzato <frederico.smozzato@gmail.com>
 Co-authored-by: Frederico Mozzato <frederico.smozzato@gmail.com>
Co-authored-by: Adoniran France <adoniranfranceh@users.noreply.github.com>
Somente o dono do projeto pode deletar o mesmo

Co-authored-by: Adoniran França <adoniranfranca@gmail.com>
Garante que somente um usuário logado e autorizado pode deletar um projeto

Co-authored-by: Adoniran França <adoniranfranca@gmail.com>
Co-authored-by: Adoniran França <adoniranfranca@gmail.com>
Co-authored_by: Adoniran França <adoniranfranca@gmail.com>
@fredericomozzato fredericomozzato merged commit eac2cda into main Jan 22, 2024
2 checks passed
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.

[Projeto] Lider apaga projeto
3 participants