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

{Service Connector} Some improvements on command experience #22822

Merged
merged 1 commit into from
Jun 13, 2022
Merged

{Service Connector} Some improvements on command experience #22822

merged 1 commit into from
Jun 13, 2022

Conversation

houk-ms
Copy link
Contributor

@houk-ms houk-ms commented Jun 10, 2022

Related command

az spring connection list-support-types
az {source} connection validate

Description

This PR fixes the following issues

  • az spring connection list-support-types doesn't work

The PR also provides the following improvements

  • Support customizing auth info when using --new parameter
  • az {source} connection validate output improvement, only show useful info

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature


This checklist is used to make sure that common guidelines for a pull request are followed.

@houk-ms houk-ms requested a review from xfz11 June 10, 2022 02:08
Comment on lines +1086 to +1094
RESOURCE.SignalR: [
CLIENT_TYPE.Dotnet,
CLIENT_TYPE.Blank
],
RESOURCE.WebPubSub: [
CLIENT_TYPE.Java,
CLIENT_TYPE.Dotnet,
CLIENT_TYPE.Blank
],
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@xfz11 please confirm the clientType here

Copy link
Member

Choose a reason for hiding this comment

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

WebPubSub also includes Python, Nodejs

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Python, Nodejs are alreay added for webapp+webpubsub

this is for Spring-Cloud, spring-cloud itself doesn't support Python NodeJs, right

@yonzhan
Copy link
Collaborator

yonzhan commented Jun 10, 2022

Service Connector

@yonzhan yonzhan requested a review from kairu-ms June 10, 2022 03:12
@yonzhan yonzhan added this to the Jun 2022 (2022-07-05) milestone Jun 10, 2022
@kairu-ms kairu-ms merged commit 7188f99 into Azure:dev Jun 13, 2022
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.

4 participants