Skip to content

DEVDOCS-6212:[Update]B2B Refresh - Non-Catalyst Channel Setup #861

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
May 1, 2025
Merged

DEVDOCS-6212:[Update]B2B Refresh - Non-Catalyst Channel Setup #861

merged 3 commits into from
May 1, 2025

Conversation

bc-Vince
Copy link
Contributor

@bc-Vince bc-Vince commented Mar 13, 2025 •
edited by jira bot
Loading

DEVDOCS-6212

What changed?

  • Article title reflects that steps are for non-Catalyst frameworks
  • Link to API Accounts article in Step 1 uses standard URL instead of directing to specific text
  • Includes an introduction with a callout to Catalyst's B2B-specific documentation
  • Step 6 now has an accompanying screenshot of the Headless Storefronts area of the B2B Edition control panel

Release notes draft

  • The article now reflects that it is geared toward non-Catalyst frameworks, and provides a link to Catalyst-specific instructions for integration.

Anything else?

ping @bc-terra

bc-terra
bc-terra previously approved these changes Mar 17, 2025
View reviewed changes
Copy link
Contributor

@bc-terra bc-terra left a comment

Choose a reason for hiding this comment

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

Looks great!

@bc-Vince
Copy link
Contributor Author

@CNanninga Can you please review this PR for content accuracy?

@bc-Vince
DEVDOCS-6212
7725ecb 
Applied the following revisions from an external review on the ticket:

* Added a paragraph calling out that the instructions are specifically for channel setup.
* Added a link to the b2b-buyer-portal repository for instructions on using the open source Buyer Portal.
* Removed callout about the invalid Vercel template.
bc-terra
bc-terra previously approved these changes Apr 25, 2025
View reviewed changes
Copy link
Contributor

@bc-terra bc-terra left a comment

Choose a reason for hiding this comment

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

lgtm

@bc-Vince
Broken URL and Article Structure Problems with B2B Headless Docs
a66e5b4 
Updated markdown for the buyer portal repository name in order to resolve a comment.
@bc-Vince bc-Vince merged commit c3e76e1 into main May 1, 2025
3 checks passed
@bc-Vince bc-Vince deleted the bc-Vince-patch-1 branch May 1, 2025 16:13
@markcmurphy markcmurphy requested a review from Copilot May 2, 2025 17:38
Copilot
Copilot AI reviewed May 2, 2025
View reviewed changes
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

Files not reviewed (1)
  • docs/b2b-edition/headless.mdx: Language not supported

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@github-actions github-actions[bot] github-actions[bot] left review comments

Copilot code review Copilot Copilot left review comments

@CNanninga CNanninga CNanninga approved these changes

@bc-terra bc-terra bc-terra left review comments

@bc-juliantran bc-juliantran Awaiting requested review from bc-juliantran

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@bc-Vince @CNanninga @bc-terra