Skip to content
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

Review connectors' name letter case. #1566

Open
YanaSegal opened this issue Dec 12, 2023 · 6 comments
Open

Review connectors' name letter case. #1566

YanaSegal opened this issue Dec 12, 2023 · 6 comments
Assignees
Labels
kind:enhancement New feature or request

Comments

@YanaSegal
Copy link

The DevEx team sent a review request to keep the product consistent.
Based on their suggestion, I listed all the connectors that need letter case updates.

As a rule, in the middle of a sentence:

  • Brand names should be capitalized
  • Connector always capitalized.
  • Inbound/outbound should be lowercase.
  • Element types should be lowercase
    for example: GitHub Webhook Intermediate Catch Event Connector should be GitHub webhook intermediate catch event Connector

List of connectors that require change:

  • MongoDB inbound Connector
  • File Watch inbound Connector
  • Email inbound Connector
  • Postgres inbound Connector
  • NATS inbound Connector
  • REST API polling

Outbound Connectors - need to be lowercase: "outbound"
image
image
image
image
image

Start events:
1st word/brand name stays capitalized, Connector stays capitalized, all the rest need to be lowercase - follow this example:
GitHub Webhook Intermediate Catch Event Connector should be GitHub webhook intermediate catch event Connector
image
Screenshot 2023-12-12 at 15 12 33
image

Intermediate event:
1st word/brand name stays capitalized, Connector stays capitalized, all the rest need to be lowercase - follow this example:
GitHub Webhook Intermediate Catch Event Connector should be GitHub webhook intermediate catch event Connector
image
image

@YanaSegal YanaSegal added the kind:enhancement New feature or request label Dec 12, 2023
@YanaSegal YanaSegal changed the title REview connectors' name letter case. Review connectors' name letter case. Dec 12, 2023
@sbuettner
Copy link
Contributor

Before doing so @YanaSegal are we sure its consistent? Here are some examples for element names from the WebModeler:

Bildschirmfoto 2023-12-13 um 08 31 46 Bildschirmfoto 2023-12-13 um 08 32 09 Bildschirmfoto 2023-12-13 um 08 34 19

@YanaSegal
Copy link
Author

@sbuettner thanks for raising that. @christinaausley, can you please review this ticket and see if we are going in the right direction? Thank you.

@christinaausley
Copy link

Hello! The Modeler team is handling adjusting these UI references, so lowercase "start event" and "intermediate catch event", etc, is how it should be written.

@YanaSegal YanaSegal self-assigned this Jan 29, 2024
@sbuettner
Copy link
Contributor

@YanaSegal Please re-open if this still an issue.

@YanaSegal YanaSegal reopened this Feb 14, 2025
@YanaSegal
Copy link
Author

Hi, yes, it is still an issue - you just need to prioritize it.

@christinaausley
Copy link

This is similar to https://github.com/camunda/developer-experience/issues/393 -- @mesellings @YanaSegal can we hop on a call to discuss this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants