-
Notifications
You must be signed in to change notification settings - Fork 33
Company Management Documentation #619
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
Conversation
fbe673a to
083a1b7
Compare
12610e8 to
e66c2d9
Compare
| |---|---|---|---| | ||
| | `creditHistoryParams` | `GetCompanyCreditHistoryParams` | No | Optional parameters for credit history filtering and pagination | | ||
| | `showCreditHistory` | `boolean` | No | Whether to show the credit history section | | ||
| | `creditHistoryParams` | `GetCompanyCreditHistoryParams` | No | Optional parameters for credit history filtering and pagination. | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
EDIT: You know, as I read the examples I realized they should not be visible to the end user, as it conflicts with how the container should be the one deciding what's filtered/etc based on the user's interaction. Keeping it brief is the right call 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@KevinBKozan
Hey Kevin, which examples are you referring to?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Specifically examples for what creditHistoryParams should look like; it's very much not worth adding it though
a620617 to
6778566
Compare
KEEPING (infrastructure + merged PRs): - Purchase Order dropin docs (merged via PR #620) - B2B Checkout extension docs - ALL enrichment files (104 files) - master source - Scripts, templates, sidebar config REMOVED (open PRs - docs live in PR branches): - Company Management docs (PR #619 open) - Company Switcher docs (PR open) - Quote Management docs (PR #622 open) - Requisition List docs (PR #623 open) - Merchant block docs (separate PRs) Workflow: enrichments in BOTH release + PR branches - Release branch = master source for ALL enrichments - PR branches = their dropin's enrichments + generated docs
26 B2B enrichment JSON files were accidentally deleted during merges. Restored from releases/b2b-nov-release.
These files are now in a dedicated branch for review: - Root-level workflow documentation - Enrichment guide files This reduces noise in dropin-specific PRs.
89a292f to
7a125a5
Compare
Removed documentation for dropins pending PR review: - company-management (PR #619 pending) - company-switcher (PR pending) - quote-management (PR #621 pending) - requisition-list (PR #622 pending) Keeping only: - purchase-order (PR #620 merged and approved) - All enrichment files (master source for generators) - All infrastructure (scripts, templates, configs)
|
Closing in favor of #654 which has a cleaner file structure (17 files instead of 580+) for easier review. All content has been preserved and verified. |
Company Management Documentation
Complete documentation for the Company Management B2B drop-in.
📖 Preview
Full preview with all dropins: https://commerce-docs.github.io/microsite-commerce-storefront/dropins-b2b/company-management/
📁 Files Added
✅ Review Checklist
Part of #600 - B2B Documentation Initiative