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

[FEATURE] First setup docker and first impressions #2

Merged
merged 40 commits into from
May 21, 2020

Conversation

alexmaramaldo
Copy link
Collaborator

@alexmaramaldo alexmaramaldo commented May 9, 2020

Por favor, não fazer o MERGE ainda, estou trabalhando em mais coisas! Criei uma TAG WIP para identificar isso!

@alexmaramaldo alexmaramaldo added the WIP PR on development yet! label May 9, 2020
@alanpbandeira
Copy link

@alexmaramaldo ví que vc utilizou o FlaskRestplus como framework pra API e, infelizmente, o projeto está abandonado e não é mais mantido (noirbizarre/flask-restplus#769).

Uma outra organização fez um Fork dele que está ativo, o Flask-restx (https://github.com/python-restx/flask-restx). Sugiro fazer essa alteração no framework, como é um Fork creio que seja questão de alterar algumas linhas de import somente

@@ -0,0 +1,20 @@
# pylint: disable=missing-module-docstring
from flask import Flask
from flask_sqlalchemy import SQLAlchemy
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tenho quase certeza que a gente vai ter que usar outro ORM. Eu acho que o Alchemy é muito manual pra gente (apesar de gostar dele). Vou ver se consigo propor alguma coisa essa semana.

from flask import request
from flask_restplus import Resource

from ..util.dto import UserDto
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Eu não conhecia esse pacote. Ele usa notations como forma de rotear os métodos do controller, é isso?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Exato, so pra ajudar a galera do front, não gosto de usar muito ele não porque deixa algumas coisas amarradas no futuro, mas por hora, quebra um galho!

@alexmaramaldo alexmaramaldo added ready and removed WIP PR on development yet! labels May 14, 2020
@alexmaramaldo
Copy link
Collaborator Author

alexmaramaldo commented May 14, 2020

@alexmaramaldo ví que vc utilizou o FlaskRestplus como framework pra API e, infelizmente, o projeto está abandonado e não é mais mantido (noirbizarre/flask-restplus#769).

Uma outra organização fez um Fork dele que está ativo, o Flask-restx (https://github.com/python-restx/flask-restx). Sugiro fazer essa alteração no framework, como é um Fork creio que seja questão de alterar algumas linhas de import somente

@alanpbandeira
Feito jovem!

@bragadennis bragadennis merged commit c311fd4 into develop May 21, 2020
@alex-girao alex-girao deleted the feature/first-setup-docker-and-first-impressions branch May 27, 2020 19:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants