Skip to content

[WIP] Update flask 3.0.0#210

Open
samuelveigarangel wants to merge 3 commits intoscieloorg:masterfrom
samuelveigarangel:update-flask-3.0.0
Open

[WIP] Update flask 3.0.0#210
samuelveigarangel wants to merge 3 commits intoscieloorg:masterfrom
samuelveigarangel:update-flask-3.0.0

Conversation

@samuelveigarangel
Copy link
Contributor

O que esse PR faz?

Fale sobre o propósito do pull request como por exemplo: quais problemas ele soluciona ou quais features ele adiciona.

Onde a revisão poderia começar?

Indique o caminho do arquivo e o arquivo onde o revisor deve iniciar a leitura do código.

Como este poderia ser testado manualmente?

Estabeleça os passos necessários para que a funcionalidade seja testada manualmente pelo revisor.

Algum cenário de contexto que queira dar?

Indique um contexto onde as modificações se fazem necessárias ou passe informações que contextualizam
o revisor a fim de facilitar o entendimento da funcionalidade.

Screenshots

Quando aplicável e se fizer possível adicione screenshots que remetem a situação gráfica do problema que o pull request resolve.

Quais são tickets relevantes?

Indique uma issue ao qual o pull request faz relacionamento.

Referências

Indique as referências utilizadas para a elaboração do pull request.

FilterNotLike,
)
from flask_admin.contrib.mongoengine.tools import parse_like_term
from flask_admin.contrib.pymongo.tools import parse_like_term
Copy link
Member

Choose a reason for hiding this comment

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

@samuelveigarangel (sugestão) talvez fosse interessante criar um módulo (opac_flask_admin) onde há todas as importações de flask_admin, por exemplo:

# conteúdo do módulo opac_flask_admin.db
from flask_admin.contrib.pymongo.tools import parse_like_term

então nos módulos o padrão ficaria:

from opac_flask_admin.db import parse_like_term

De modo que quando vier a ocorrer mudanças no flask_admin, a troca fica mais suave.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants