[IMP] l10n_in_ewaybill_stock: add documentation for Ewaybill on stock #12353
Conversation
|
|
There was a problem hiding this comment.
Hi @hamo-odoo!Thank you for your work on this Indian localization page :)
I added some comments/suggestions. Please take a look at it and let me know if you have any questions.
Thanks!
hamo-odoo
force-pushed
the
17.0-ewaybill-stock-doc-hamo
branch
2 times, most recently
from
aa0b47c to
7a7ca76
Compare
|
@afma-odoo Thank you for suggestion and making it look more clean and nice. I have made the suggested changes |
hamo-odoo
force-pushed
the
17.0-ewaybill-stock-doc-hamo
branch
2 times, most recently
from
3a74750 to
19eb6a7
Compare
There was a problem hiding this comment.
Hi @hamo-odoo! Thanks for the changes ;)
I just have some comments and suggestions. For lines 307-309, I'm not sure which fields to fill and what information to include in those fields. Can you specify it?
Thank you!
hamo-odoo
force-pushed
the
17.0-ewaybill-stock-doc-hamo
branch
2 times, most recently
from
d65cc95 to
d739afb
Compare
|
@afma-odoo Changes done |
There was a problem hiding this comment.
Thank you @hamo-odoo for updating the page!
I add the be-doc-review for the next step ;)
hamo-odoo
changed the title
hamo-odoo
force-pushed
the
17.0-ewaybill-stock-doc-hamo
branch
2 times, most recently
from
a5055e3 to
eb0574d
Compare
hamo-odoo
changed the title
There was a problem hiding this comment.
Hello @hamo-odoo thank you for your work! I added a few comments and suggestions 🙂
content/applications/finance/fiscal_localizations/india/e-waybill-stock-report.png
Outdated
Show resolved
Hide resolved
hamo-odoo
force-pushed
the
17.0-ewaybill-stock-doc-hamo
branch
from
eb0574d to
d314abc
Compare
hamo-odoo
force-pushed
the
17.0-ewaybill-stock-doc-hamo
branch
3 times, most recently
from
3f0af97 to
c13db9f
Compare
|
@auva-odoo Can you please checkout the changes? |
hamo-odoo
force-pushed
the
17.0-ewaybill-stock-doc-hamo
branch
from
c13db9f to
dbaf0b3
Compare
auva-odoo
force-pushed
the
17.0-ewaybill-stock-doc-hamo
branch
from
dbaf0b3 to
f9b0c10
Compare
There was a problem hiding this comment.
@hamo-odoo As discussed, I pushed some changes 🙂 Could you review them to ensure everything looks good and remains accurate? You can r+ yourself if it does (and when all checks have passed 😉 )
@robodoo delegate+
task-4574445 closes #12353 Signed-off-by: Harsh Modi (hamo) <[email protected]>
task-4574445