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

Update taxes docs #847

Merged
merged 9 commits into from
Jul 27, 2023
Merged

Update taxes docs #847

merged 9 commits into from
Jul 27, 2023

Conversation

peelar
Copy link
Member

@peelar peelar commented Jul 26, 2023

Features

  • split up the Taxes documentation into separate articles (the provider pages will be extended in a separate PR)
  • added description for general features (e.g. tax code matcher) and Avatax features (e.g. disabling document recording)

This PR is a stepping stone towards establishing a more general Apps template & providing the structure for later extending the Avatax configuration guide (in a separate PR).

@peelar peelar requested a review from a team as a code owner July 26, 2023 09:15
@peelar peelar self-assigned this Jul 26, 2023
@vercel
Copy link

vercel bot commented Jul 26, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
saleor-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 27, 2023 2:35pm

@peelar
Copy link
Member Author

peelar commented Jul 26, 2023

saleor/apps#808 - must be merged first

@lkostrowski lkostrowski changed the title update taxes docs WIP update taxes docs Jul 26, 2023
@peelar peelar added the blocked label Jul 26, 2023
@lkostrowski lkostrowski changed the title WIP update taxes docs [Do not merge] update taxes docs Jul 26, 2023
@peelar
Copy link
Member Author

peelar commented Jul 27, 2023

@saleor/appstore @lkostrowski extracted the feature descriptions into small guides. I also changed the structure more towards the new agreed app docs structure. Please have a look.

@peelar peelar requested a review from lkostrowski July 27, 2023 13:05
@peelar peelar changed the title [Do not merge] update taxes docs Update taxes docs Jul 27, 2023
@peelar peelar removed the blocked label Jul 27, 2023
@peelar peelar merged commit 0640b68 into main Jul 27, 2023
2 checks passed
@peelar peelar deleted the update-taxes-docs branch July 27, 2023 14:38
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.

2 participants