Skip to content

Conversation

@samueljlieber
Copy link
Contributor

@samueljlieber samueljlieber commented May 6, 2024

Task: #3809629

Adds Canada l10n accounting doc to the fiscal localizations.

@robodoo
Copy link
Collaborator

robodoo commented May 6, 2024

@samueljlieber
Copy link
Contributor Author

8ae1252 Added preliminary content outlined in this task to the following sections:

  • Modules installation
  • Chart of accounts
  • Fiscal positions
  • Taxes
  • AvaTax
  • Reports
  • Cash discount

@samueljlieber
Copy link
Contributor Author

95d6b7d Added a section for Payroll @larm-odoo

@larm-odoo
Copy link
Contributor

95d6b7d Added a section for Payroll @larm-odoo

Hi Sam - per Catalina (cas): "I wouldn't add Payroll to it because we don't have any type of localized workflow for CA. If a Canadian customer would like to know more about Payroll the general doc should be enough"

So for now, I will not add that to the doc. If anything changes, I'll be sure to add the necessary info!

@samueljlieber samueljlieber force-pushed the 17.0-accounting-l10n-ca-sali branch from 95d6b7d to b9adb68 Compare September 6, 2024 19:46
@samueljlieber
Copy link
Contributor Author

b9adb68

  • Expanded on introduction
  • added Canadian-specific report information
  • expanded on writing checks
  • removed payroll section
  • expanded on receiving pre-authorized debits (stripe)

@samueljlieber samueljlieber force-pushed the 17.0-accounting-l10n-ca-sali branch from b9adb68 to 1066a2d Compare September 12, 2024 13:09
@samueljlieber
Copy link
Contributor Author

1066a2d

  • Listed default fiscal positions
  • Added screenshot of fiscal positions

@samueljlieber samueljlieber marked this pull request as ready for review September 12, 2024 13:09
@C3POdoo C3POdoo requested a review from a team September 12, 2024 13:12
Copy link
Contributor

@StraubCreative StraubCreative left a comment

Choose a reason for hiding this comment

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

Hi @samueljlieber
Looks good to me! I added a couple small comments for you to consider, however we can move to the next round with @odoo/localizations-doc-review for the sake of deployment. Thank you for putting this together!

Co-authored-by: Zachary Straub <zst@odoo.com>
@samueljlieber samueljlieber force-pushed the 17.0-accounting-l10n-ca-sali branch from 7b5d946 to 3956fe4 Compare September 12, 2024 21:12
@samueljlieber
Copy link
Contributor Author

Thank your for your helpful review @StraubCreative! I implemented your suggestions in 3956fe4.

@samueljlieber
Copy link
Contributor Author

@robodoo r+

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