-
Notifications
You must be signed in to change notification settings - Fork 38
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
Epic 1748 doc ref handling #3361
base: main
Are you sure you want to change the base?
Conversation
* wip (cherry picked from commit a57e940) * wip (cherry picked from commit 5634aea) * prepare end-to-end draft (cherry picked from commit 92ec79f) * prepare end-to-end draft (cherry picked from commit 5a808bd) * rework deserializers (cherry picked from commit 0b0a558) * incorporate feedback, move jackson code into a separate module * clean up accidental change * naming adjustments * lint * remove extra dollar sign * remove extra dollar sign
…different package (#3149) * refactor: move document stuff to a different module --------- Co-authored-by: Jonathan Roques <[email protected]>
#2976) * feat(doc-handling): Some investigations to use documents in the REST connector
this( | ||
secretProvider, | ||
validationProvider, | ||
new DocumentFactoryImpl(InMemoryDocumentStore.INSTANCE), |
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.
I think we should always expect this parameter to be passed in.
return new CamundaDocument( | ||
camundaDocumentReference.metadata(), camundaDocumentReference, documentStore); | ||
} | ||
if (reference instanceof ExternalDocumentReference ignored) { |
Check notice
Code scanning / CodeQL
Unread local variable Note
|
||
boolean matches(DocumentOperation operationReference); | ||
|
||
Object execute(DocumentOperation operationReference, Document document); |
Check notice
Code scanning / CodeQL
Useless parameter Note
🚀 Deployment Results 🚀Please find below the results of the latest deployments.
|
Description
Related issues
closes #
Checklist
no milestone
label.