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

[feature request] Adicionar view de registro #54

Merged
merged 4 commits into from
Nov 6, 2023
Merged

[feature request] Adicionar view de registro #54

merged 4 commits into from
Nov 6, 2023

Conversation

HladczukLe
Copy link
Collaborator

@HladczukLe HladczukLe commented Oct 31, 2023 •
edited by bitterteriyaki
Loading

O que este PR introduz?

Closes #29.

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

@bitterteriyaki bitterteriyaki changed the title Feature/register view [feature request] Adicionar view de registro Oct 31, 2023
@bitterteriyaki bitterteriyaki added feature request New feature request. review needed This PR will be reviewed in the future. labels Oct 31, 2023
@bitterteriyaki bitterteriyaki added this to the Sprint 3 milestone Oct 31, 2023
bitterteriyaki
bitterteriyaki suggested changes Oct 31, 2023
View reviewed changes
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.

As views não podem retornar None, e evite utilizar a sintaxe A | B visto que ela quebra a compatibilidade com versões mais antigas do Python.

apps/users/views.py Outdated Show resolved Hide resolved
apps/users/views.py Outdated Show resolved Hide resolved
server/settings/base.py Outdated Show resolved Hide resolved
bitterteriyaki
bitterteriyaki suggested changes Oct 31, 2023
View reviewed changes
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.

Esta atualização volta com o bug que acontece no menu dropdown do usuário.

image

bitterteriyaki
bitterteriyaki approved these changes Nov 6, 2023
View reviewed changes
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.

LGTM.

@bitterteriyaki bitterteriyaki merged commit 9dcc0c3 into main Nov 6, 2023
1 check passed
@bitterteriyaki bitterteriyaki deleted the feature/register-view branch November 6, 2023 23:43
@bitterteriyaki bitterteriyaki removed the review needed This PR will be reviewed in the future. label Nov 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@bitterteriyaki bitterteriyaki bitterteriyaki approved these changes

Assignees

@HladczukLe HladczukLe

Labels
feature request New feature request.
Projects
Squad 06 - Project Board
Status: ✅ Feito
Milestone
Sprint 3
Development

Successfully merging this pull request may close these issues.

[feature request] Criar view de registro de usuário
2 participants
@HladczukLe @bitterteriyaki