Replies: 1 comment 3 replies
-
I think the javadoc needs to be updated and aligned with the code in the UpdateClaimJob. That said, I will add comments to your propositions below:
I wouldn't say that the new logic invalidates what the job does. The job#updateClaim does 2 things: link the claim with a service and increment attemps and update status. With the new logic, the job will still check if service is available and if a credential exists and modify the claim in consequence. I'm not sure to follow what you mean. |
Beta Was this translation helpful? Give feedback.
3 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
From #154, users need to select the service from a popup in the new claims form.
Therefore, this new logic invalidates what the claiming service job is meant to do: search the services that have been requested in a claim using the "serviceRequested" property (something like "postgres-8"). This is because now we can directly link services to claim when creating a new one.
Further clarification is required here to know how to approach this situation:
Beta Was this translation helpful? Give feedback.
All reactions