-
Notifications
You must be signed in to change notification settings - Fork 39
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
Update template descriptions and add keywords to improve refactorings suggestions #3089
Comments
@chillleader @Oleksiivanov @igpetrov What is the source of truth for element template descriptions? I couldn't find anything other than the templates themselves? |
@philippfromme depends where:
|
Yes, I'm talking about the templates themselves. But since they are generated I was wondering where the descriptions are coming from and who is responsible for them. |
The template creator is responsible for providing the description. |
@johnBgood is this something you could pick up and discuss with @philippfromme? @philippfromme is happy to help and hopefully, this would be a quick pick to get done, it's going to improve the connector suggestions. |
@philippfromme It's generated using the Element Template Generator (ETG). The ETG parse our Java models and generates the json files. For instance in the JdbcRequestData record. Please let me know if it's clear enough, I'm happy to discuss it more. |
Okay, I finally found what I was looking for. The descriptions are in the source files: https://github.com/search?q=repo%3Acamunda%2Fconnectors%20description%20%3D&type=code. For example: connectors/connectors/jdbc/src/main/java/io/camunda/connector/jdbc/outbound/JdbcFunction.java Line 26 in fcc8705
And to add the new |
I think you missed my last message :'( |
@johnBgood Sorry, I did actually see your message and based on it I was able to find what I was looking for, so thank you. 😃 |
@johnBgood Would it look something like this? |
@philippfromme I adjusted the PR a bit, you will find how the element templates for the JDBC Connector have been updated. Let me know if it's what you wanted :) |
@johnBgood Thanks for helping out with that! |
@johnBgood For connectors like the Asana connector there seems to be a template only. Are these not generated and therefore have to be modified directly? |
@philippfromme exactly, some connectors are based on other connectors (the REST connector in this case). You can see it looking at these lines: {
"type": "Hidden",
"value": "io.camunda:http-json:1",
"binding": {
"type": "zeebe:taskDefinition",
"property": "type"
}
}, http-json:1 is the REST Connector. Just putting this here for context, you're right you need to update the JSON file directly for these connectors. |
@johnBgood Another question: In some cases (e.g. AWS Bedrock) there JSON templates and Java files (one JSON, on hybrid JSON and a JAVA file). But in the web modeler there is only one template when I try to use them. Do I need to adjust all of them or are some of them generated? |
@philippfromme If there are Java files, then the JSON will be generated. But only in our repo. So yea, in this example you need to update:
|
But this going to change soon if I'm not mistaken. For now I'll focus on the PR and we'll take it from there. |
True :) |
What should we do?
The ✨ feature for selected elements in Web Modeler suggests connectors based on an element's name. The suggestions are based on the name and description of the connector templates and therefore the quality of suggestions depends on those.
The latest version of the ✨ also takes into account keywords that were added to a template. For example, the following template
has a
keywords
field with relevant keywords. If an element's name mentions replying to a tweet the connector will be suggested based on the matching keyword.Why should we do it?
To improve the quality of suggestions.
Related to bpmn-io/refactorings#27
The text was updated successfully, but these errors were encountered: