l10n-brazil icon indicating copy to clipboard operation
l10n-brazil copied to clipboard

[12.0] l10n_br_fiscal need_validation

Open rvalyi opened this issue 3 years ago • 5 comments

Uma limitação no modulo fiscal atual é que qualquer usuário do fiscal tem que preencher campos como fiscal_type, icms_origin, tax_icms_or_issqn e ncm_id sempre que criar um novo produto. Mas tem algumas situaçoes onde isso não é legal, como:

  1. usuario de compra que ta comprando um produto novo do qual ele ainda desconhece os parametros fiscais (que poderiam vir na importação do XML da NFe mais tarde por examplo).
  2. usuario fazendo o inventario: precisa levantar o inventario e pode deixar os parametros fiscais para ajustar depois. Um usuario de estoque pode nao ser usuario fiscal (e não enxergar esses campos então) mas se ele for isso pode ser um problema.

A ideia é que temos aqui um novo grupo como "usuario fiscal leniente"/"Fiscal Laxist User". Produtos (e tb parceiros) tem agora um flag need_fiscal_validation (inspirado pelo modulo base_tier_validation que poderia ser plugado por cima caso houver necessidade). Por padrão quando um usuário do grupo usuário fiscal leniente criar um novo produto ou usuario, esse flag need_fiscal_validation vem como True. Isso permite uma filtragem dos cadastros para uma revisão ulterior.

E por fim, se alguém tantar validar um documento fiscal que tiver um parceiro ou um produto precisando de validação fiscal, ai o Odoo bloca, reclamando que tem que validar os cadastros primeiro.

O que vcs acham? Tb nao seria impossivel empacotar isso como modulo separado, nao sei qual ficaria melhor...

rvalyi avatar Aug 26 '21 14:08 rvalyi

Hi @rvalyi! Thank you very much for this contribution. As the addon you are improving does not have a declared maintainer, I take the opportunity to mention that you can consider adopting it. To do so, please read the maintainer role description, and, if interested, create a pull request to add your GitHub login to the maintainers key of the addon manifest.

OCA-git-bot avatar Aug 26 '21 14:08 OCA-git-bot

Acho super valido :D isso realmente enche o saco muitas vezes, logo vamos revisar a pr tbm

antoniospneto avatar Aug 26 '21 14:08 antoniospneto

valeu pelo review @felipemotter eu fiz as correçoes, se puder revisar de novo eu agradeço.

cc @marcelsavegnago @netosjb @renatonlima @mbcosta

rvalyi avatar Oct 08 '21 04:10 rvalyi

Desculpa a demora para revisar @rvalyi , estávamos no meio de uma implementação e as coisas estavam corridas. Ficou bem fluído e já dá pra usar facilmente em produção, porém achamos alguns detalhes que acredito serem passiveis de melhora:

  1. Um usuário fiscal manager, caso precisasse revisar um produto no momento que estivesse fazendo uma fatura ou cotação, não encontraria o campo need_validation, pois ele está sendo colocado apenas no form do modelo product.template, mas nas linhas da fatura/cotação é usado o form do modelo product.product. adm-criar-sem-need-validation

  2. Quando um fiscal user tenta criar um item diretamente pela fatura, nenhuma informação fiscal é mostrada e ainda é acusado um erro devido a falta dessas informações. Não é o fluxo ideal para produção, mas já que o odoo permite, acho válido manter a funcionalidade e evitar o erro. PS: Não tem a ver diretamente com o código da PR, mas envolve o mesmo escopo. Caso queira, podemos fazer uma PR com a correção. criar-produto-sem-fiscal

Resumão: propomos replicar as informações no form do model product.template para o do product.product.

felipemotter avatar Oct 15 '21 01:10 felipemotter

Pois é, essa é uma dificuldades que nos passamos com o mateus quando estavamos fazendo o mission do marketing-tools. Nossa venda da missão é baseada em um product.product que usa o product.template. Também achei interessante sua proposição @felipemotter

Em qui., 14 de out. de 2021 às 22:53, felipemotter @.***> escreveu:

Desculpa a demora para revisar @rvalyi https://github.com/rvalyi , estávamos no meio de uma implementação e as coisas estavam corridas. Ficou bem fluído e já dá pra usar facilmente em produção, porém achamos alguns detalhes que acredito serem passiveis de melhora:

Um usuário fiscal manager, caso precisasse revisar um produto no momento que estivesse fazendo uma fatura ou cotação, não encontraria o campo need_validation, pois ele está sendo colocado apenas no form do modelo product.template, mas nas linhas da fatura/cotação é usado o form do modelo product.product. [image: adm-criar-sem-need-validation] https://user-images.githubusercontent.com/6812128/137418719-59236ef6-be47-4205-b525-f01a0a830fdc.gif 2.

Quando um fiscal user tenta criar um item diretamente pela fatura, nenhuma informação fiscal é mostrada e ainda é acusado um erro devido a falta dessas informações. Não é o fluxo ideal para produção, mas já que o odoo permite, acho válido manter a funcionalidade e evitar o erro. PS: Não tem a ver diretamente com o código da PR, mas envolve o mesmo escopo. Caso queira, podemos fazer uma PR com a correção. [image: criar-produto-sem-fiscal] https://user-images.githubusercontent.com/6812128/137419353-f9c17421-08f5-4e2b-979a-7310d99c7099.gif

Resumão: propomos replicar as informações no form do model product.template para o do product.product.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/OCA/l10n-brazil/pull/1592#issuecomment-943918903, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJEVVIG2S7GVON2LZKQDMLDUG6CRJANCNFSM5C3OFWCA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

marcos-mendez avatar Oct 15 '21 02:10 marcos-mendez

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

github-actions[bot] avatar Jul 16 '23 12:07 github-actions[bot]

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

github-actions[bot] avatar Nov 19 '23 12:11 github-actions[bot]