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

Test-AzSignalRName - Increase Clarity #25792

Closed
jpomfret opened this issue Aug 9, 2024 · 3 comments · Fixed by #25827
Closed

Test-AzSignalRName - Increase Clarity #25792

jpomfret opened this issue Aug 9, 2024 · 3 comments · Fixed by #25827
Labels
customer-reported Service Attention This issue is responsible by Azure service team. SignalR

Comments

@jpomfret
Copy link

jpomfret commented Aug 9, 2024

Type of issue

Missing information

Feedback

Test-AzSignalRName - the use of this command is not very clear from the docs.

Looking at the code this makes sense, but from reading the docs it isn't clear that this is checking whether a name is available for use.

At first read I was expecting this to return true if the name did exist, so the results felt backwards.

Also the wording of the examples is not super clear.

Example 1: Check an existed name
Example 2: Check an unexisted name.

Maybe these could be rewritten to something like:

Example 1: Check if a name is already in use
Example 2: Check if a name is available to use.

Page URL

https://learn.microsoft.com/en-gb/powershell/module/az.signalr/test-azsignalrname?view=azps-12.2.0

Content source URL

https://github.com/Azure/azure-powershell/blob/main/src/SignalR/SignalR/help/Test-AzSignalRName.md

Author

@mikefrobbins

Document Id

f0d3d385-215a-21a6-109d-c42f6faf2c92

@jpomfret jpomfret added the needs-triage This is a new issue that needs to be triaged to the appropriate team. label Aug 9, 2024
@isra-fel isra-fel added Service Attention This issue is responsible by Azure service team. SignalR and removed needs-triage This is a new issue that needs to be triaged to the appropriate team. labels Aug 15, 2024
Copy link
Contributor

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @sffamily, @chenkennt.

@chenkennt
Copy link
Member

@Y-Sindo please help take a look

@jpomfret
Copy link
Author

Thanks folks! Much clearer 💜

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
customer-reported Service Attention This issue is responsible by Azure service team. SignalR
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants