search

OCA / l10n-brazil

Localização brasileira oficial do Odoo.
https://odoo-community.org/psc-teams/brazil-66
GNU Affero General Public License v3.0
241 stars 245 forks source link

[14.0][IMP] l10n_br_fiscal: force workflow transtition #3130

Open mileo opened 4 months ago

OCA-git-bot commented 4 months ago

Hi @renatonlima, some modules you are maintaining are being modified, check this out!

antoniospneto commented 4 months ago

eu adaptei a mesma funcionalidade no #3081 mas um pocuo diferente, adicionei o parametro force_change na função _change_state

@mileo será que não é melhor lidar com isso da forma que eu fiz ? eu penso que o usar o contexto fica muito implicito, menos transparente..

mileo commented 4 months ago

eu adaptei a mesma funcionalidade no #3081 mas um pocuo diferente, adicionei o parametro force_change na função _change_state

@mileo será que não é melhor lidar com isso da forma que eu fiz ? eu penso que o usar o contexto fica muito implicito, menos transparente..

O contexto pode ser chamado de qualquer outro modelo sem alterar a chamada da sub função, eu prefiro. Fiz isso para poder confirmar uma fatura de um cliente que saiu do passo do workflow não entendi o pq ainda.

Referente ao seu PR eu não revisei ainda, capaz que hoje de tempo.

antoniospneto commented 4 months ago

eu adaptei a mesma funcionalidade no #3081 mas um pocuo diferente, adicionei o parametro force_change na função _change_state @mileo será que não é melhor lidar com isso da forma que eu fiz ? eu penso que o usar o contexto fica muito implicito, menos transparente..

O contexto pode ser chamado de qualquer outro modelo sem alterar a chamada da sub função, eu prefiro. Fiz isso para poder confirmar uma fatura de um cliente que saiu do passo do workflow não entendi o pq ainda.

Referente ao seu PR eu não revisei ainda, capaz que hoje de tempo.

ah entendi, qualquer coisa dá pra fazer uma combinação e permitir as duas formas.

Essa questão das iconsistencias do status talvez seja o problema que o Felipe corrigiu aqui: https://github.com/OCA/l10n-brazil/pull/3093