Skip to content
This repository has been archived by the owner on May 28, 2023. It is now read-only.

Gostaria de poder requisitar datasets usando <org ou usuário>/<nome do dataset> #72

Open
JoseRenan opened this issue Oct 30, 2019 · 4 comments
Labels
discussão Further information is requested melhoria Nova funcionalidade ou sugestão

Comments

@JoseRenan
Copy link
Member

JoseRenan commented Oct 30, 2019

Descrição da melhoria:
Atualmente não é possível fazer requisições do tipo: GET /datasets/opendevufcg/roadmap-cc, é impossível criar datasets com nome "opendevufcg/roadmap-cc" porque / é caractere reservado pra URLs, porém é interessante suportar pra termos uma abordagem de usuário e organização como o github faz

Como ela pode ser implementada:
Ao invés da requisição ter a URL /datasets/:name, teríamos /datasets/:org/:name só fica a discussão se seria bom quebrar em dois no json também 🤔 se sim, teríamos de passar essa modificação pra CLI tbm (o que não é um problema já que não tem nada em produção)

@JoseRenan JoseRenan added discussão Further information is requested melhoria Nova funcionalidade ou sugestão labels Oct 30, 2019
@juliobguedes
Copy link

Acredito que quebrar em /:org/:name facilite tanto na organização dos dados, quanto no sentido de distribuir o laguinho: teríamos um service e um conjunto de pares chave-valor, onde a chave seria o nome da organização e o valor onde deve ser buscada essa informação.

Ainda assim, no contexto de se manter na URI atual, é possível definir algum caractere separador para dividir a org e o repositório aleatório, solucionando o problema

@JoseRenan
Copy link
Member Author

@juliobguedes, o problema não é nem como fica a URI, mas se deveria ser armazenado no banco o nome como "opendevufcg/roadmap" ou quebrar em 2 atributos mesmo, pq no caso de quebrar, isso interferiria no publish na CLI, se a gente muda só como a URI se comporta, fica tudo certo na CLI, mas eu acho que seria melhor quebrar em dois mesmo 🤔

@fanny
Copy link
Contributor

fanny commented Oct 30, 2019

Eu acho que é melhor salvar os dois isolados, por conta daquilo de ser o mais atômico1 possível e acho que é mais fácil nomear os atributos se eles forem separados. No momento, nāo consigo ver uma vantagem de salvar assim a url, mas a título de informaçāo, o github salva das duas formas, o nome da pessoa, e o repo separados, e eles dois juntos, que1 ele chama de nameWithOwner talvez seja o caso de tentar ver onde isso é usado e ver se faria sentido pra gnt tb

@fanny
Copy link
Contributor

fanny commented Oct 30, 2019

https://developer.github.com/v4/object/repository lá no fim, nāo consegui referenciar, é algo como:

    "id": 1296269,
    "node_id": "MDEwOlJlcG9zaXRvcnkxMjk2MjY5",
    "name": "Hello-World",
    "nameWithOwner": "octocat/Hello-World",
    "owner": {
      ....

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
discussão Further information is requested melhoria Nova funcionalidade ou sugestão
Projects
None yet
Development

No branches or pull requests

3 participants