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

Integrar publish com laguinho-api #21

Closed
JoseRenan opened this issue Sep 8, 2019 · 7 comments · Fixed by #23
Closed

Integrar publish com laguinho-api #21

JoseRenan opened this issue Sep 8, 2019 · 7 comments · Fixed by #23
Assignees
Labels
enhancement New feature or request

Comments

@JoseRenan
Copy link
Member

Pessoas!!!! como pudemos ver no OpenDevUFCG/laguinho-api#45 , habemus publish no backend, fazendo com que agora possamos concluir a parte que não coloquei na #12 que era a de enviar o dataset para o laguinho-api.

Quem pegar essa issue deve submeter os metadados do dataset que serão publicados para o POST /datasets do laguinho API.

As informações de como rodar o servidor e etc constam no repositório da API.

@JoseRenan JoseRenan added the enhancement New feature or request label Sep 8, 2019
@lucasaarcoverde
Copy link
Contributor

Oii, posso fazer?

@JoseRenan
Copy link
Member Author

Pode sim!!

@lucasaarcoverde
Copy link
Contributor

Estava resolvendo a issue e notei que a API_URL precisa ser global, tanto para o get utilizar quanto para o publish, mas não sei uma forma "legal" de colocar globalmente, alguma dica?

@JoseRenan
Copy link
Member Author

Eu sinceramente não tinha pensado nisso ainda kkkkkkkkkkk tipo, o que geralmente é feito é botar variável de ambiente, mas a questão é que pode ter uma release do laguinho instalada na minha máquina e ela começar a usar a URL que tá na variável de ambiente de desenvolvimento... daí tem de pensar em alguma coisa pra resolver isso 🤔

@JoseRenan
Copy link
Member Author

/cc @OpenDevUFCG/laguinho

@JoseRenan
Copy link
Member Author

JoseRenan commented Sep 9, 2019

Talvez seja uma solução considerar que sempre que usarmos o laguinho-dev está rodando em desenvolvimento, o laguinho-dev executa a CLI pelo main, enquanto a versão de release vai rodar a partir do setup.py, daí a gente pode definir alguma flag com isso, no main a gente bota uma variável global DEVELOPMENT pra true, e daí significa que ele deve procurar uma variável de ambiente pra definir qual a global URL 🤔 nusei acho que isso resolve

@lucasaarcoverde
Copy link
Contributor

Massa, vou tentar fazer isso então. qualquer coisa pergunto 💃

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants