Skip to content

Conversation

@eapa-odoo
Copy link
Contributor

Revise the India localization documentation to ensure it aligns with the updated
functionality in the master branch.

ref: odoo/odoo#190265

@robodoo
Copy link
Collaborator

robodoo commented Feb 13, 2025

Pull request status dashboard

@C3POdoo C3POdoo requested a review from a team February 13, 2025 07:35
@afma-odoo afma-odoo requested review from afma-odoo and removed request for a team February 13, 2025 15:32
@eapa-odoo eapa-odoo force-pushed the master-l10n_in-doc-eapa branch from aeaf764 to b76616d Compare February 14, 2025 13:22
Copy link
Contributor

@afma-odoo afma-odoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @eapa-odoo! Thank you for your work on this Indian localization page! I added some comments/suggestions.
As mentioned in our documentation guidelines, we usually tend to use images sparingly, only when necessary, to avoid the need for frequent updates with new versions. Additionally, the content should be understandable even without the image. Screenshots should also be resized, as they would not be readable on lower-resolution screens. Don't hesitate to refer to our documentation guidelines for help.
Please take a look at it and let me know if you have any questions. Thanks!

@eapa-odoo eapa-odoo force-pushed the master-l10n_in-doc-eapa branch from b76616d to 6013f48 Compare February 18, 2025 11:03
@eapa-odoo
Copy link
Contributor Author

Hello @afma-odoo,

Thank you for reviewing my PR and for your suggestions.

I have implemented the requested changes, and since the e-Invoice feature has been updated, I've adjusted that as well.

Could you please review it again?

Thanks! 😊

Copy link
Contributor

@afma-odoo afma-odoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello @eapa-odoo,
Thank you for the update. I approved it, could you just resize the screenshot in the "E-way bill cancellation" section? Thanks a lot! 🙂

@eapa-odoo eapa-odoo force-pushed the master-l10n_in-doc-eapa branch from 6013f48 to cc8ee63 Compare February 20, 2025 09:56
@eapa-odoo
Copy link
Contributor Author

Hello, @afma-odoo
I have updated the screenshot, can you please have a look?

Thanks 😀

@afma-odoo
Copy link
Contributor

Hi @eapa-odoo!
Thank you, everything's fine now 😉

@afma-odoo afma-odoo requested a review from a team February 21, 2025 09:45
Revise the India localization documentation to ensure it aligns with the updated
functionality in the master branch.

ref: odoo/odoo#190265
@eapa-odoo eapa-odoo force-pushed the master-l10n_in-doc-eapa branch from cc8ee63 to b953c98 Compare February 25, 2025 06:42
Copy link
Contributor

@auva-odoo auva-odoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

robodoo pushed a commit that referenced this pull request Feb 25, 2025
Revise the India localization documentation to ensure it aligns with the updated
functionality in the master branch.

closes #12058

Ref: odoo/odoo#190265
Signed-off-by: Audrey Vandromme (auva) <auva@odoo.com>
@robodoo robodoo added the 18.3 label Feb 25, 2025
@robodoo robodoo closed this Feb 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants