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

Address feedback from UX studies for 5.1.0-beta.x #11970

Closed
3 tasks
deyaaeldeen opened this issue Oct 21, 2020 · 1 comment · Fixed by #14086
Closed
3 tasks

Address feedback from UX studies for 5.1.0-beta.x #11970

deyaaeldeen opened this issue Oct 21, 2020 · 1 comment · Fixed by #14086

Comments

@deyaaeldeen
Copy link
Member

deyaaeldeen commented Oct 21, 2020

  • Add visual to the documentation to explain the mined opinions structure and naming better. i.e. Small graphic with opinion, sentence etc. in sample/
  • Consider improving the docstrings for all the Options bags to include what type of options are there.
  • Adding helper method to get opinions? Is helper method for optional property feasible?

Based on Azure/azure-sdk-for-net#15809.

Epic: #13792

@deyaaeldeen deyaaeldeen added this to the [2020] November milestone Oct 21, 2020
@deyaaeldeen deyaaeldeen self-assigned this Oct 21, 2020
openapi-sdkautomation bot pushed a commit to AzureSDKAutomation/azure-sdk-for-js that referenced this issue Dec 15, 2020
@deyaaeldeen
Copy link
Member Author

Update:

@ghost ghost closed this as completed in #14086 Mar 3, 2021
ghost pushed a commit that referenced this issue Mar 3, 2021
By adding a line to the `analyzeSentiment` docstring about OM. Fixes #11970 after I talked to @maririos that this should be enough for now.
@github-actions github-actions bot locked and limited conversation to collaborators Apr 12, 2023
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants