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

[communication] add some tnm tests #11628

Merged
merged 4 commits into from
Oct 5, 2020
Merged

[communication] add some tnm tests #11628

merged 4 commits into from
Oct 5, 2020

Conversation

0rland0Wats0n
Copy link
Contributor

No description provided.

Copy link
Member

@DominikMe DominikMe left a comment

Choose a reason for hiding this comment

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

Nicely done, left three comments.

@@ -133,6 +133,7 @@ export type GetAreaCodesOptions = OperationOptions;
// @public
export interface GetAreaCodesRequest {
countryCode: string;
locationOptionsQueries: LocationOptionsQueries;
Copy link
Member

Choose a reason for hiding this comment

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

// @public
export interface LocationOptionsQuery {
    labelId?: string;
    optionsValue?: string;
}

^ those two props shouldn't be optional right? Another bug.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yep another one

Copy link
Member

@DominikMe DominikMe left a comment

Choose a reason for hiding this comment

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

PhoneNumber REST APIs need more work regarding the LocationOptions.
This PR is a good step forward 👍

And tests are great.

@0rland0Wats0n 0rland0Wats0n merged commit 6b2bc1a into Azure:feature/communication/phoneNumber Oct 5, 2020
openapi-sdkautomation bot pushed a commit to AzureSDKAutomation/azure-sdk-for-js that referenced this pull request Jan 4, 2021
add automation track2 config (Azure#11628)

* add automation track2 config

* update config
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.

2 participants