-
Notifications
You must be signed in to change notification settings - Fork 14
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
#1997 - Upgrade Zeebe node version #2031
Conversation
…GRPC resource exhauted error.
Kudos, SonarCloud Quality Gate passed!
|
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,, nice work @dheepak-aot
@@ -86,7 +87,7 @@ export interface AssessmentConsolidatedData { | |||
studentDataPartnerStudyWeeks?: number; | |||
studentDataPartnerEmploymentInsurance?: YesNoOptions; | |||
studentDataPartnerFedralProvincialPDReceiptCost?: number; | |||
studentDataParentDependentTable?: unknown; | |||
studentDataParentDependentTable?: JSONDoc; |
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.
just to check, should this be a list of JSONDoc
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.
May be. But we never populated data to these values in tests, so kept it as JSONDoc. When we create an interface to this value in future we can update to list of that interface that extends JSONDoc.
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 👍 . Added a minor qn.
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.
Nice catch, looks good 👍
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.
Great job! Looks good!
Update Zeebe node version
createProcessInstanceWithResult()
of Zeebe client.Note
Initial effort was to upgrade to Zeebe 8.2.4 latest version of zeebe node (https://www.npmjs.com/package/zeebe-node?activeTab=versions) but upgrading to latest version brought back the GRPC resource exhausted error logs.
Hence upgrading with second latest/penultimate version 8.2.3.