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

Business Cards returns Appearance.Style.Name == print #18216

Closed
maririos opened this issue Jan 27, 2021 · 1 comment · Fixed by #19999
Closed

Business Cards returns Appearance.Style.Name == print #18216

maririos opened this issue Jan 27, 2021 · 1 comment · Fixed by #19999
Assignees
@maririos
Labels
Client This issue points to a problem in the data-plane of the library. Cognitive - Form Recognizer
Milestone
[2021] April

Comments

@maririos
Copy link
Member

maririos commented Jan 27, 2021 •
edited
Loading

The Business Cards endpoint is returning print as an Appearance.Style.Name value. And although it is designed as an extensible enum and this doesn't brake the SDK it seems like a service bug.
Once fixed, respective tests need to be enabled.

-> https://github.com/Azure/azure-sdk-for-net/blob/master/sdk/formrecognizer/Azure.AI.FormRecognizer/tests/SessionRecords/FormRecognizerClientLiveTests/StartRecognizeBusinessCardsIncludeFieldElements.json#L216
@maririos maririos added Client This issue points to a problem in the data-plane of the library. Cognitive - Form Recognizer labels Jan 27, 2021
@maririos maririos added this to the [2021] March milestone Jan 27, 2021
@maririos
Copy link
Member Author

Service identified this as a bug in 2.1-preview.2 version.
It only happens in BusinessCards when IncludeFieldElements=true and it has been fixed in 2.1-preview.3.
The SDK will surface the value as is and when it moves to target 2.1-preview.3 bug will be fixed. -> enable commented validation again

@maririos maririos modified the milestones: [2021] March, [2021] April Feb 22, 2021
@maririos maririos self-assigned this Mar 31, 2021
@maririos maririos mentioned this issue Mar 31, 2021
Merged
openapi-sdkautomation bot pushed a commit to AzureSDKAutomation/azure-sdk-for-net that referenced this issue Mar 22, 2022
@zjpjack @matthchr
CodeGen from PR 18324 in Azure/azure-rest-api-specs
d9fea6e 
Review request for Microsoft.ContainerService to add version 2022-02-01 (Azure#18324)

* Adds base for updating Microsoft.ContainerService from version stable/2022-01-01 to version 2022-02-01

* Updates readme

* Updates API version in new specs and examples

* Update readmes for the 2022-02-01 dev branch of container service (Azure#17887)

* update readme

* remove useless -only tags

* Fix violated rule R2026 for 2022-02-01 managedCluster swagger (Azure#18024)

* fix allof

* add missing type

* GA alias minor version (Azure#18038)

* GA alias minor version

* polish more details on kubernetesVersion and currentKubernetesVersion

* Update specification/containerservice/resource-manager/Microsoft.ContainerService/stable/2022-02-01/managedClusters.json

minor revision

Co-authored-by: Matthew Christopher <[email protected]>

Co-authored-by: Matthew Christopher <[email protected]>

* add missing type object (Azure#18115)

* Revert "GA alias minor version (Azure#18038)" (Azure#18291)

This reverts commit 3a99cda3cb48e05c74923f5467c0737014322b0f.

* Align modifications of several common definitions with 2022-02-02-preview for container service 2022-02-01 (Azure#18216)

* sync modification

* fix ref

Co-authored-by: Jianping Zeng <[email protected]>
Co-authored-by: Matthew Christopher <[email protected]>
@github-actions github-actions bot locked and limited conversation to collaborators Mar 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@maririos maririos

Labels
Client This issue points to a problem in the data-plane of the library. Cognitive - Form Recognizer
Projects
None yet
Milestone
[2021] April
Development

Successfully merging a pull request may close this issue.

[FR] Enable tests maririos/azure-sdk-for-net
1 participant
@maririos