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

Hint for process ID is missing in 1.0.0 #693

Open
barmac opened this issue May 30, 2022 · 3 comments
Open

Hint for process ID is missing in 1.0.0 #693

barmac opened this issue May 30, 2022 · 3 comments
Labels
backlog Queued in backlog bug Something isn't working

Comments

@barmac
Copy link
Member

barmac commented May 30, 2022

Describe the Bug

There used to be a description for process ID for Camunda 7:

image

It's gone in 1.0.0:

image

Steps to Reproduce

  1. do this
  2. do that
  3. now this happens

Expected Behavior

There should be a hint.

Environment

  • Host (Browser/Node version), if applicable: [e.g. MS Edge 18, Chrome 69, Node 10 LTS]
  • OS: [e.g. Windows 7]
  • Library version: [e.g. 2.0.0]
@barmac barmac added the bug Something isn't working label May 30, 2022
@MaxTru
Copy link
Contributor

MaxTru commented May 30, 2022

@ingorichtsmeier could you please comment on this issue and elaborate why it makes sense to have this hint added?

@MaxTru MaxTru added the help wanted Extra attention is needed label May 30, 2022
@nikku
Copy link
Member

nikku commented Jun 10, 2022

The reason you want to have it is that what is called ID BPMN wise becomes the KEY deployment (run-time) wise and runtime process definition IDs are generated instead.

Not sure if this is mitigated in C8, but I guess for C7 we should just re-instantiate it. Or await a repeat of the feedback we got in the past.

@nikku nikku added ready Ready to be worked on and removed help wanted Extra attention is needed labels Jun 10, 2022
@MaxTru MaxTru self-assigned this Jun 21, 2022
@MaxTru
Copy link
Contributor

MaxTru commented Jun 21, 2022

Different to what was expected: this is not a 5 minute fix since this should appear for C7 but not C8. However the ID Properties are C7 / C8 agnostic. So we need to overwrite IDProps for C7.

Moving to backlog and also marking for spring cleaning.

@MaxTru MaxTru added backlog Queued in backlog and removed ready Ready to be worked on labels Jun 21, 2022
@MaxTru MaxTru removed their assignment Jun 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog Queued in backlog bug Something isn't working
Development

No branches or pull requests

3 participants