[12.0][FIX] account_financial_report: Computed hierarchy get groups from account code#4
Conversation
carlosdauden
force-pushed
the
12.0-IMP-account_financial_report-forgeflow-computed_hierarchy
branch
from
03690ee to
4a21781
Compare
|
¿Por qué no vale el nivel 3/4 de jerarquía? Creo que es complicar un poco el código para una necesidad que se puede cubrir con lo otro, ¿no? |
|
Vale, veo una explicación en la tarea asociada, pero sigo creyendo que sobrefactorizar esto por el capricho de sacarlo a 4 dígitos... que lo saquen a nivel 3, que es lo mismo. |
carlosdauden
force-pushed
the
12.0-IMP-account_financial_report-forgeflow-computed_hierarchy
branch
from
4a21781 to
4459093
Compare
|
Realmente no se trata de 3 o 4, sino de poder elegir libremente el número de dígitos a agrupar. Esta es una funcionalidad básica que tiene cualquier programa de contabilidad que puedes comprar por 50€ en un supermercado. 😜 |
|
Entonces quitemos otras opciones? Tú qué piensas, @JordiBForgeFlow ? |
carlosdauden
force-pushed
the
12.0-IMP-account_financial_report-forgeflow-computed_hierarchy
branch
6 times, most recently
from
26cfef8 to
e61fbeb
Compare
carlosdauden
force-pushed
the
12.0-IMP-account_financial_report-forgeflow-computed_hierarchy
branch
from
e61fbeb to
d6805c7
Compare
carlosdauden
changed the title
2 participants
@Tecnativa