Skip to content

Conversation

@Skaiir
Copy link
Contributor

@Skaiir Skaiir commented Sep 25, 2025

Proposed Changes

It is currently impossible to determine, without looking into the diagram, what element template is missing from your diagram on load. This is a particularly glaring issue when importing in marketplace in web modeler SM, if you haven't configured connectors.

Kinda related to: camunda/camunda-modeler#5109, I found this UX problem last minute so this is a super quick patch. Discussion is here: https://camunda.slack.com/archives/C097RLZP0MB/p1758793244155529.

Before:

image image

After:

image

Checklist

To ensure you provided everything we need to look at your PR:

  • Brief textual description of the changes present
  • Visual demo attached
  • Steps to try out present, i.e. using the @bpmn-io/sr tool
  • Related issue linked via Closes {LINK_TO_ISSUE} or Related to {LINK_TO_ISSUE}

@bpmn-io-tasks bpmn-io-tasks bot added the needs review Review pending label Sep 25, 2025
@barmac barmac requested review from a team, barmac and philippfromme and removed request for a team September 25, 2025 12:05
@philippfromme
Copy link
Contributor

What's the state of this PR? @Skaiir told me that there was some pushback in regards to this PR. Should it be a draft again?

@nikku
Copy link
Member

nikku commented Oct 2, 2025

@Skaiir What user journey does this support? How will the user be able to "move forward" once they have the ID in their hands? I'm struggling to understand.

What I'd do instead:

  • Be able to extend the menu, accounting for application context
    • Web Modeler can add "Search template on marketplace"
    • User is able to install from market place
    • Problem resolves

Copy link
Member

@nikku nikku left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If we go ahead with this change:

  • Let's ensure id and version are indicated => This is what is matched against
  • Consider to make id and version code blocks; they are technical identifiers

Also, consider my earlier feedback.

I'd love to ship real solutions, no quick fixes.

@bpmn-io-tasks bpmn-io-tasks bot added in progress Currently worked on and removed needs review Review pending labels Oct 2, 2025
@nikku
Copy link
Member

nikku commented Oct 2, 2025

If we consider the existing UX for applied and found templates:

image

Why don't we adapt it to show the relevant information (id and version) in case of not found?

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

in progress Currently worked on

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants