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.

Sign up for GitHub

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

Jump to bottom

Azure Text Analytics Readme Issue #11490

Closed
zzhxiaofeng opened this issue Sep 25, 2020 · 2 comments · Fixed by #11583
Closed

Azure Text Analytics Readme Issue #11490

zzhxiaofeng opened this issue Sep 25, 2020 · 2 comments · Fixed by #11583
Labels
bug This issue requires a change to an existing behavior in the product in order to be resolved. Client This issue points to a problem in the data-plane of the library. Cognitive - Text Analytics Docs good first issue This issue tracks work that may be a good starting point for a first-time contributor test-manual-pass

Comments

@zzhxiaofeng
Copy link
Contributor

Section link
Screenshot 2020-09-23 155502
Reason:
Wrong property name.

Suggestion:
Update to entity.confidenceScore.

@jongio for notification.
@ghost ghost added the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Sep 25, 2020
@tzhanl tzhanl added Client This issue points to a problem in the data-plane of the library. Cognitive - Text Analytics Docs needs-team-triage Workflow: This issue needs the team to triage. test-manual-pass labels Sep 25, 2020
@ghost ghost removed the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Sep 25, 2020
@ramya-rao-a
Copy link
Contributor

Thanks for reporting @zzhxiaofeng

@deyaaeldeen, Do you agree with the recommendation above? If not, then please provide your recommendation after which

  • add the labels bug, help wanted and good first issue
  • remove the label needs-team-triage
  • assign the issue to the backlog milestone

@deyaaeldeen deyaaeldeen added bug This issue requires a change to an existing behavior in the product in order to be resolved. good first issue This issue tracks work that may be a good starting point for a first-time contributor and removed needs-team-triage Workflow: This issue needs the team to triage. labels Sep 25, 2020
@deyaaeldeen deyaaeldeen added this to the Backlog milestone Sep 25, 2020
@deyaaeldeen
Copy link
Member

Thanks @zzhxiaofeng for reporting and you are indeed correct, that needs to be fixed as you described.

@deyaaeldeen deyaaeldeen mentioned this issue Oct 1, 2020
Merged
openapi-sdkautomation bot pushed a commit to AzureSDKAutomation/azure-sdk-for-js that referenced this issue Nov 11, 2020
@paulgmiller @bcho
@levimm @mainred @norshtein @bowang-666
CodeGen from PR 11588 in Azure/azure-rest-api-specs
2b1b466 
Dev containerservice microsoft.container service 2020 11 01 (Azure#11588)

* Adds base for updating Microsoft.ContainerService from version stable/2020-09-01 to version 2020-11-01

* Updates readme

* Updates API version in new specs and examples

* add autoupgradeprofile to 2020-11-01 (Azure#11363)

* add autoupgradeprofile

* add autoupgrader profile to mc

* format change

Co-authored-by: Xiahe Liu <Xiahe.Liu@microsoft.com>

* add pod subnet id on agentpool (Azure#11310)

* aks: add pod identity profile spec (Azure#11366)

* add private dns zone property in 2020-11-01 (Azure#11311)

Co-authored-by: Li Ma <lima2@microsoft.com>

* Remove invalid pattern and adjust description (Azure#11312)

* aks: fix pod identity profile field names (Azure#11484)

* aks: add missing `podIdentityProfile.enabled` field

* aks: typo

* add non upgrade channel (Azure#11492)

Co-authored-by: Xiahe Liu <Xiahe.Liu@microsoft.com>

* Make identityProfile mutable (Azure#11458)

* Add KubeletConfig and LinuxOSConfig specs (Azure#11490)

* aks: update readme tag for 1101 (Azure#11472)

* aks: update readme tag

* aks: add `2020-11-01-only`

* resovle merge conlicts to master, adding recent master change to dev branch (Azure#11604)

Co-authored-by: Xiahe Liu <Xiahe.Liu@microsoft.com>

Co-authored-by: Xiahe Liu <Xiahe.Liu@microsoft.com>
Co-authored-by: Paul Miller <pmiller@microsoft.com>
Co-authored-by: hbc <me@hbc.rocks>
Co-authored-by: Super <mali_no2@hotmail.com>
Co-authored-by: Li Ma <lima2@microsoft.com>
Co-authored-by: Qingchuan Hao <haoqingchuan@hotmail.com>
Co-authored-by: Tongyao Si <tosi@microsoft.com>
Co-authored-by: Bo Wang <61758179+bowang-666@users.noreply.github.com>
Co-authored-by: Phoenix He <tih@microsoft.com>
openapi-sdkautomation bot pushed a commit to AzureSDKAutomation/azure-sdk-for-js that referenced this issue Nov 13, 2020
CodeGen from PR 11490 in Azure/azure-rest-api-specs
6d4bf93 
Add KubeletConfig and LinuxOSConfig specs (Azure#11490)
@xirzec xirzec removed this from the Backlog milestone May 17, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Apr 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug This issue requires a change to an existing behavior in the product in order to be resolved. Client This issue points to a problem in the data-plane of the library. Cognitive - Text Analytics Docs good first issue This issue tracks work that may be a good starting point for a first-time contributor test-manual-pass
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Text Analytics] Fix README.md deyaaeldeen/azure-sdk-for-js
5 participants
@xirzec @deyaaeldeen @ramya-rao-a @zzhxiaofeng @tzhanl