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

ACP-2128: Updated Vertex documentation #2430

Merged
merged 9 commits into from
Jan 15, 2024

Conversation

bohdan-turchyk-spryker
Copy link
Contributor

@bohdan-turchyk-spryker bohdan-turchyk-spryker commented Jan 9, 2024

PR Description

Checklist

  • I agree with the Code Contribution License Agreement in CONTRIBUTING.md

@matweew

This comment was marked as resolved.

@matweew matweew changed the title Updated Vertex documentation ACP-2128: Updated Vertex documentation Jan 9, 2024
@bohdan-turchyk-spryker bohdan-turchyk-spryker added the updated PR contains meaningful changes to some document label Jan 10, 2024
matweew
matweew previously approved these changes Jan 10, 2024
matweew
matweew previously approved these changes Jan 10, 2024

## Next step
[Configure Vertex in the Back Office](/docs/pbc/all/tax-management/{{page.version}}/base-shop/third-party-integrations/vertex/configure-vertex.html)
[Configure Vertex in the Back Office](/docs/pbc/all/tax-management/{{page.version}}/base-shop/third-party-integrations/vertex/configure-vertex.html)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've kept this "next step" section here as it is a next step after installation is finished; at the same time the same section is present in "integrate-vertex-app.md" file. Should we keep it like this or should it be present in both files?

Also: should we keep the folder structure like this or should new files be moved lower in the hierarchy?

@lenadoc lenadoc merged commit 0c1b785 into master Jan 15, 2024
13 checks passed
@lenadoc lenadoc deleted the feature/acp-2128/tax-app-stack-update branch January 15, 2024 17:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
TW review needed PR needs Technical writer's review updated PR contains meaningful changes to some document
Development

Successfully merging this pull request may close these issues.

3 participants