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

Implementar requisição a API #5

Open
3 tasks
itepifanio opened this issue Aug 23, 2019 · 3 comments
Open
3 tasks

Implementar requisição a API #5

itepifanio opened this issue Aug 23, 2019 · 3 comments
Labels
enhancement New feature or request question Further information is requested

Comments

@itepifanio
Copy link
Member

Feature

Da forma que testei o backbone:

import utils
from odufrn_api_py import ODUFRNApi
odufrn_api = ODUFRNApi('aplicação', 'segredo', 'xapikey', 'v1')
print(odufrn_api._make_requests('https://api.info.ufrn.br/acao-associada/v1/oportunidades-bolsas').content)

A requisição está sendo feita manualmente (passando toda a url). Deve-se adicionar um método que faça a requisição do recurso. Note que existe um desafio aqui, existem url de vários formatos, ex:

  • https://api.info.ufrn.br/curso/v1/componentes-curriculares?disciplina-obrigatoria=true
  • https://api.info.ufrn.br/curso/v1/componentes-curriculares/1/programas
  • https://api.info.ufrn.br/curso/v1/componentes-curriculares/

Ainda não sei bem como implementar essa requisição de recurso (podemos discutir isso).

  • Tipo: Nova implementação

Checklist:

  • Implementar feature;
  • Adicionar testes;
  • Atualizar docstring;
@itepifanio itepifanio added enhancement New feature or request question Further information is requested labels Aug 23, 2019
@johnazedo
Copy link
Member

Utilizando o .content é necessário transformar em string para não dar erro no python3.5

Exemplo:

odufrn_api._make_requests('https://api.info.ufrn.br/acao-associada/v1/oportunidades-bolsas').content.decode('utf-8')

@johnazedo
Copy link
Member

Acredito que esse problema é muito grande para ser resolvida com em uma issue . Sugiro dividir ela em mais issues. Além de ser preciso de uma mini reunião para discutirmos como resolver.

@itepifanio
Copy link
Member Author

Acredito que esse problema é muito grande para ser resolvida com em uma issue . Sugiro dividir ela em mais issues. Além de ser preciso de uma mini reunião para discutirmos como resolver.

Tá com a tag questions pra gente discutir como será feito mesmo. Concordo que essa issue será fechada e aberta outras.

@alvarofpp @nymarya @diegodiogenes vocês tem alguma ideia de como seguir com a implementação?

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
Projects
None yet
Development

No branches or pull requests

2 participants