l10n-brazil
l10n-brazil copied to clipboard
[12.0] l10n_br_fiscal need_validation
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:
- 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).
- 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...
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.
Acho super valido :D isso realmente enche o saco muitas vezes, logo vamos revisar a pr tbm
valeu pelo review @felipemotter eu fiz as correçoes, se puder revisar de novo eu agradeço.
cc @marcelsavegnago @netosjb @renatonlima @mbcosta
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:
-
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.
-
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.
Resumão: propomos replicar as informações no form do model product.template para o do product.product.
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.
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.
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.