-
Notifications
You must be signed in to change notification settings - Fork 225
fix: add contentHash to document properties #7454
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
Conversation
|
👋 🤖 ✅ Looks like the changes were ported across versions, nice job! 🎉 You can read more about the versioning within our docs in our documentation guidelines. |
|
@giorgionaps very small change, but please just give it a TW review 🙏 |
giorgionaps
left a comment
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.
No comments from me
|
@barmac can you please add the relevant labels - it helps us triage PRs and make sure we don't miss anything 🙏 |
|
OK done! I've also added the missing part for the |
|
Thanks @barmac - @giorgionaps please TW re-review so we can get this released (assuming it is okay from a technical pov) 🙏 |
giorgionaps
left a comment
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.
No comments from me on this one
Description
The documentation is missing the contentHash property which is required to retrieve the stored response. But in fact, the property is set on the document variable. This PR fixes the missing documentation.
When should this change go live?
bugorsupportlabel)available & undocumentedlabel)holdlabel)low priolabel)PR Checklist
{type}(scope): {description}commit message(s)/docsdirectory (version 8.9)./versioned_docsdirectory.@camunda/tech-writersunless working with an embedded writer.