Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[IMP] accounting/l10n_uy: clarify onboarding #11715

Closed
wants to merge 1 commit into from
Closed

Conversation

fvz-odoo
Copy link
Contributor

@fvz-odoo fvz-odoo commented Jan 8, 2025

Currently, users are having a hard time setting up their Uruware account. With the improvement to the documentation we are adding clarifications to simplify the onboarding process as well as adding a see also section so users have more documentation and smart classes/eLearning available.

@fvz-odoo fvz-odoo requested a review from samueljlieber January 8, 2025 00:39
@robodoo
Copy link
Collaborator

robodoo commented Jan 8, 2025

Pull request status dashboard

Copy link
Contributor

@samueljlieber samueljlieber left a comment

Choose a reason for hiding this comment

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

Hi @fvz-odoo! Nice work on this PR, your improvements look good to me. Just a couple suggestions and one question about Demo mode. I will push up these changes as a second commit, once you review I can squash and move this PR over to the l10n doc team for final reviews :)

@samueljlieber
Copy link
Contributor

@fvz-odoo one more question: do these changes apply to 17.0 and 18.0? or only 17.0? :)

@fvz-odoo
Copy link
Contributor Author

fvz-odoo commented Jan 8, 2025

@zaoral
Copy link

zaoral commented Jan 8, 2025

@MarianaLuzTorres

@fvz-odoo
Copy link
Contributor Author

fvz-odoo commented Jan 8, 2025

@fvz-odoo one more question: do these changes apply to 17.0 and 18.0? or only 17.0? :)

17.0 and 18.0 😉, that's why I did it in 17 so we can forward port it afterwards.

Thanks!

Fernando

@samueljlieber samueljlieber changed the title [IMP] update uruguay.rst to clarify onboarding [IMP] accounting/l10n_uy: clarify onboarding Jan 8, 2025
@samueljlieber
Copy link
Contributor

@fvz-odoo Looks good to me, I squashed our commits in 186ab8c. We can open this PR for review, if you are ready :)

@fvz-odoo fvz-odoo marked this pull request as ready for review January 8, 2025 22:10
@samueljlieber samueljlieber changed the base branch from 17.0 to 18.0 January 8, 2025 22:10
@samueljlieber samueljlieber changed the base branch from 18.0 to 17.0 January 8, 2025 22:10
@C3POdoo C3POdoo requested a review from a team January 8, 2025 22:11
Copy link
Contributor

@dade-odoo dade-odoo left a comment

Choose a reason for hiding this comment

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

Hey @fvz-odoo , thanks for updating the Uruguay localization doc! I've made some comments. Please let me know if you have any questions.
Also, we recently received this feedback regarding the localization:
https://www.odoo.com/odoo/project.task/4450335
Could you please change "valid" to "paid" in the first sentence of the section: "To set up a Uruware account requires a paid Odoo subscription".
Thanks!

@samueljlieber
Copy link
Contributor

Squashed commits and cleaned up RST line breaks in 6699d4c

@fvz-odoo
Copy link
Contributor Author

Hey @fvz-odoo , thanks for updating the Uruguay localization doc! I've made some comments. Please let me know if you have any questions. Also, we recently received this feedback regarding the localization: https://www.odoo.com/odoo/project.task/4450335 Could you please change "valid" to "paid" in the first sentence of the section: "To set up a Uruware account requires a paid Odoo subscription". Thanks!

Hey @dade-odoo actually, one app free is considered as well. We could change it to "paid" but a valid subscription is enough to create an Uruware account. 😉

Let us now if we still should change the wording on this.

Cheers!

Fernando

@samueljlieber

Currently, users are having a hard time setting up their Uruware account. With the improvement to the documentation we are adding clarifications to simplify the onboarding process as well as adding a see also section so users have more documentation and smart classes/eLearning available.

Co-authored-by: dade-odoo <[email protected]>
@dade-odoo
Copy link
Contributor

Hey @samueljlieber , I spoke with @fvz-odoo and clarified the valid vs paid subscription issue. I just edited the beginning of the "Set up a Uruware account" section to reflect that and be a little easier to read, so y'all can take a look and r+ if you're happy with it. Let me know if you need anything else from me.
Thanks!

@samueljlieber
Copy link
Contributor

@robodoo r+

robodoo pushed a commit that referenced this pull request Jan 14, 2025
Currently, users are having a hard time setting up their Uruware account. With the improvement to the documentation we are adding clarifications to simplify the onboarding process as well as adding a see also section so users have more documentation and smart classes/eLearning available.

closes #11715

Signed-off-by: Samuel Lieber (sali) <[email protected]>
Co-authored-by: dade-odoo <[email protected]>
@robodoo robodoo closed this Jan 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants