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

m365 spo homesite get requires additional siteUrl parameter for multiple home sites experience #6490

Open
reshmee011 opened this issue Nov 16, 2024 · 1 comment

Comments

@reshmee011
Copy link
Contributor

Priority

(Low) Something is a little off

Description

m365 spo homesite get returns only home site with multiple home sites experience

I have 5 home sites /Viva Connection experiences. Yet the cmdlet returns only one Home Site.

The endpoint to return all home sites is _api/SPO.Tenant/GetTargetedSitesDetails requiring refactoring of existing cmdlet.

I can work on the cmdlet once reviewed.

Steps to reproduce

m365 spo homesite get

Expected results

More than one home site should return if there are more than one home sites configured.
image

Actual results

Returns one home site
image

Diagnostics

No response

CLI for Microsoft 365 version

v10.1.0

nodejs version

v20.8.1

Operating system (environment)

Windows

Shell

PowerShell

cli doctor

No response

Additional Info

No response

@reshmee011 reshmee011 added the bug label Nov 16, 2024
@reshmee011 reshmee011 changed the title m365 spo homesite get returns only home site with multiple home sites experience m365 spo homesite get requires additional siteUrl parameter for multiple home sites experience Nov 16, 2024
@milanholemans
Copy link
Contributor

As discussed previously, it is important to note that the new option url should be an optional parameter. Since CLI requires backward compatibility, we cannot introduce breaking changes just yet. This is something we should do in v11. When the user uses this command without url option, we should still use the old endpoint and throw a warning that using this command without url is deprecated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants