search

OpenDevUFCG / laguinho-api

Onde todos os dados, pessoas e radiação da UFCG se encontram.
MIT License
28 stars 18 forks source link

Adiciona obtenção de dataset por nome #50

Closed lucasmedeiros closed 5 years ago

lucasmedeiros commented 5 years ago

Resolves #48

Esta PR cria um endpoint datasets/:nomedodataset, e retorna um JSON com um valor dataset, que será null caso não exista, ou retornará o JSON correspondente ao dataset encontrado.

Exemplos de funcionamento:

  1. Quando não há referência de um dataset com o nome informado: image

  2. Quando existe dataset com o nome informado: image

lucasmedeiros commented 5 years ago

@JoseRenan Inclusive tenho uma pergunta, tu acha que seria legível manter esse lambda que eu usei e até o operador ternário? Fiquei meio na dúvida agora...

JoseRenan commented 5 years ago

@JoseRenan Inclusive tenho uma pergunta, tu acha que seria legível manter esse lambda que eu usei e até o operador ternário? Fiquei meio na dúvida agora...

Acho que sim, na CLI a gente tem um list comprehension, então acho que é ok usar esses conceitos de programação funcional

lucasmedeiros commented 5 years ago

@JoseRenan feito!

JoseRenan commented 5 years ago

@lucasmedeiros eu to com um problema, aquele negócio do / no fim, no POST faz diferença ter ou não ter, no GET não tá fazendo diferença, a gente pode optar por duas soluções, a primeira é: adotar o padrão de que não adicionamos / no fim, por exemplo: GET /datasets vai pra listagem de tudo, GET /datasets/ vai pra o get por nome com uma string vazia, e GET /datasets/blablaba/ dá 404, ouuuu, adicionar dois @route pra cada função, um com um / no fim e outra sem

lucasmedeiros commented 5 years ago

@JoseRenan sim, vou resolver isso agora mesmo. (PS: amei que tu marcou sem querer um cara aleatório na PR KKKKKKKKK)

lucasmedeiros commented 5 years ago

@JoseRenan resolvi o problema utilizando o atributo strict_slashes, agora em todos funciona com ou sem as /.