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

Verificação dos parâmetros obrigatórios #63

Open
filipealvess opened this issue Aug 5, 2022 · 0 comments
Open

Verificação dos parâmetros obrigatórios #63

filipealvess opened this issue Aug 5, 2022 · 0 comments
Labels
enhancement New feature or request question Further information is requested refactor

Comments

@filipealvess
Copy link
Collaborator

filipealvess commented Aug 5, 2022

@felipealencar, enquanto eu tava fazendo o PR #62, percebi que é bem melhor usar decorators do que arquivos de constantes pra fazer validações.

Aí pensei de a gente atualizar esse padrão pra verificação dos parâmetros obrigatórios, ao invés de usar aquele arquivo required_params.py.

A gente poderia ter uma função parecida com a has_token do PR #62 que verificaria se a lista de parâmetros passados nela estão no corpo da requisição. Por exemplo:

@required(['email', 'password'])
def login():
  ...

Poderia até deixar o arquivo de constantes, ele seria usado na passagem dos parâmetros pra essa função, algo assim:

@required(required_params['users']['read'])
def login():
  ...

Como o arquivo de constantes já tá funcionando, não seria algo urgente, mas vê se vale a pena 💡😉

@filipealvess filipealvess added question Further information is requested enhancement New feature or request refactor labels Aug 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request question Further information is requested refactor
Projects
None yet
Development

No branches or pull requests

1 participant