Skip to content

Conversation

@Saurabh7019
Copy link
Contributor

Closes #6653

@Saurabh7019 Saurabh7019 marked this pull request as ready for review December 2, 2025 10:10
Copilot AI review requested due to automatic review settings December 2, 2025 10:10
Copy link

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.

Pull request overview

This pull request introduces a new command spo brandcenter settings list to retrieve SharePoint Brand Center configuration settings. The implementation follows the established patterns for SPO commands that accept only global options, using the Zod schema validation approach.

  • Adds a new SPO command to list Brand Center configuration
  • Includes comprehensive test coverage with schema validation tests
  • Provides complete documentation with examples and response formats in multiple output types

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
src/m365/spo/commands/brandcenter/brandcenter-settings-list.ts Main command implementation that fetches Brand Center configuration via SharePoint REST API
src/m365/spo/commands/brandcenter/brandcenter-settings-list.spec.ts Comprehensive test suite covering validation, success scenarios, and error handling
src/m365/spo/commands.ts Registers the new BRANDCENTER_SETTINGS_LIST command constant
docs/src/config/sidebars.ts Adds the new command to the documentation sidebar under brandcenter category
docs/docs/cmd/spo/brandcenter/brandcenter-settings-list.mdx Complete documentation including usage, permissions, examples, and response formats
.devproxy/api-specs/sharepoint.yaml Adds API specification for the Brand Center Configuration endpoint with security requirements


## Examples

List all brand center config
Copy link

Copilot AI Dec 2, 2025

Choose a reason for hiding this comment

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

[nitpick] The example description uses the abbreviation "config" instead of the full word "configuration" used in the command description. For consistency, consider using "Lists the brand center configuration" or "Retrieve the brand center configuration".

Suggested change
List all brand center config
List all brand center configuration

Copilot uses AI. Check for mistakes.
@Saurabh7019 Saurabh7019 marked this pull request as draft December 2, 2025 11:02
@Saurabh7019 Saurabh7019 marked this pull request as ready for review December 2, 2025 12:27
@milanholemans
Copy link
Contributor

Thanks! We'll try to review it soon.

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.

New command: spo brandcenter settings list

2 participants