Skip to content
This repository has been archived by the owner on Jan 3, 2024. It is now read-only.

[feature request] Adicionar página de perfil do usuário #82

Merged
merged 6 commits into from
Nov 22, 2023

Conversation

HladczukLe
Copy link
Collaborator

@HladczukLe HladczukLe commented Nov 19, 2023

O que este PR introduz?

Closes #64.

Checklist

Por favor, cheque se seu PR atende aos seguintes requisitos:

  • Eu atualizei a documentação de acordo (se necessário).
  • Eu adicionei testes para cobrir minhas mudanças (se necessário).
  • Todos os novos e existentes testes passaram.
  • Meu código segue o estilo de código deste projeto.

Tipo de PR

Que tipo de mudança este PR introduz?

  • Correção de bug
  • Feature request
  • Atualização na documentação
  • Atualização no estilo do código (formatação, variáveis locais, etc.)
  • Refatoração (sem mudanças funcionais, sem mudanças na API)
  • Atualizações de dependências
  • Mudanças relacionadas a CI/CD
  • Chore (mudanças não relacionadas a código)
  • Mudanças relacionadas a build (e.g. Dockerfile, pyproject.toml, etc.)

Este PR introduz uma mudança que quebra a compatibilidade?

  • Sim
  • Não

Outras informações (screenshots, detalhes, etc.)

@HladczukLe HladczukLe added feature request New feature request. review needed This PR will be reviewed in the future. labels Nov 19, 2023
@HladczukLe HladczukLe added this to the Sprint 05 milestone Nov 19, 2023
@HladczukLe HladczukLe self-assigned this Nov 19, 2023
Copy link
Contributor

@bitterteriyaki bitterteriyaki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Será necessário adicionar uma view para a página de perfis. Essa view vai receber o ID do usuário como parâmetro (pode ser o nome de usuário também, se você quiser). Com o ID (ou o nome de usuário), você vai pesquisar o usuário no banco de dados e mostrar o perfil do usuário com o ID dado.

Funções que podem ajudar:

  • get_object_or_404 (para procurar o usuário, e caso não encontre, retornar um erro 404)

server/urls.py Outdated Show resolved Hide resolved
apps/users/urls.py Outdated Show resolved Hide resolved
templates/user/profile.html Outdated Show resolved Hide resolved
Copy link

codecov bot commented Nov 21, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (9986d37) 97.65% compared to head (147a037) 97.69%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #82      +/-   ##
==========================================
+ Coverage   97.65%   97.69%   +0.04%     
==========================================
  Files          33       33              
  Lines         341      347       +6     
==========================================
+ Hits          333      339       +6     
  Misses          8        8              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@bitterteriyaki bitterteriyaki temporarily deployed to unb-mds-feature-user-pa-b19jzb November 21, 2023 22:54 Inactive
@bitterteriyaki bitterteriyaki temporarily deployed to unb-mds-feature-user-pa-mqbmpo November 22, 2023 03:12 Inactive
@bitterteriyaki bitterteriyaki temporarily deployed to unb-mds-feature-user-pa-mqbmpo November 22, 2023 03:18 Inactive
Copy link

codeclimate bot commented Nov 22, 2023

Code Climate has analyzed commit 147a037 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 97.6% (0.0% change).

View more on Code Climate.

@bitterteriyaki bitterteriyaki merged commit 5a01521 into main Nov 22, 2023
4 checks passed
@bitterteriyaki bitterteriyaki deleted the feature/user-page branch November 22, 2023 03:25
@bitterteriyaki bitterteriyaki removed the review needed This PR will be reviewed in the future. label Nov 24, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature request New feature request.
Projects
Status: ✅ Feito
Development

Successfully merging this pull request may close these issues.

[feature request] Adicionar página de perfil do usuário
2 participants