-
Notifications
You must be signed in to change notification settings - Fork 188
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
docs: add textract doc #4239
docs: add textract doc #4239
Conversation
👋 🤖 🤔 Hello! Did you make your changes in all the right places? These files were changed only in docs/. You might want to duplicate these changes in versioned_docs/version-8.5/.
You may have done this intentionally, but we wanted to point it out in case you didn't. You can read more about the versioning within our docs in our documentation guidelines. |
@DenovVasil Do you need a technical writer review for this work? |
yes but after the engineers review. I will ping you when I get approvals from the engineers |
fdc86c5
to
46688f2
Compare
To store the **Text** from the first block in a variable `lineText`, the **Confidence** in `textConfidence`, and the **BlockType** in `blockType`, use the following result **expression**: | ||
|
||
```feel | ||
= {lineText: blocks[2].text, textConfidence: blocks[2].confidence, blockType: blocks[2].blockType} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With provided example it should take first aray block :
= {lineText: blocks[2].text, textConfidence: blocks[2].confidence, blockType: blocks[2].blockType}
= {lineText: blocks[2].text, textConfidence: blocks[2].confidence, blockType: blocks[2].blockType} | |
= {lineText: blocks[0].text, textConfidence: blocks[0].confidence, blockType: blocks[0].blockType} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is a real example, and in my case index:2 contains all the necessary values
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Don't forget to add this documentation to the versioned_docs directory if needed, and add label to this PR with the release version. Move it to 'Ready for Review' and consider adding the Connector teams as reviewers, as well as @camunda/tech-writers as the technical reviewer
46688f2
to
6727651
Compare
@mesellings Can you pleas make a technical writer review ? |
@DenovVasil I can review this for you 👍 |
8ceeec8
Sorry @mesellings I didn't see this. Please of course feel free to have a look over this -- four eyes are better than two. I just made a handful of grammatical adjustments anyways. |
@DenovVasil I will review this today - I'll probably make some commits directly to the branch as well 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@DenovVasil Lgtm - I have made some edits as follows:
- Restructuring and heading levels
- Rewording for clarity, active voice etc.
Happy to approve - great work 👍
@DenovVasil Is this ready to be merged for the 8.6 release next week? |
5a1d3c4
to
5f39915
Compare
🧹 Preview environment for this PR has been torn down. |
* docs: add textract doc
Description
Add Textract documentation
dev ticket
epic
When should this change go live?
hold
label or convert to draft PR)PR Checklist
/versioned_docs
directory./docs
directory (aka/next/
).