Skip to content

fix(create-nodes): allow setting the values of elements in createnodes - #363

Merged
JoernT merged 2 commits into
devfrom
create-nodes-child-elements
Jul 29, 2026
Merged

fix(create-nodes): allow setting the values of elements in createnodes#363
JoernT merged 2 commits into
devfrom
create-nodes-child-elements

Conversation

@DrRataplan

Copy link
Copy Markdown
Collaborator

And refactor a bit, process more as path steps, and see both attributes and elements as value tests

Fixes #362

What does this change?

This fixes #362 by setting values for elements if the path refers to them

How was this tested?

  • Unit tests (npm test) pass
  • Relevant Cypress e2e tests pass, or a new demo exercises the change
  • Linting (npm run lint) passes

Related issues

Closes #362

@DrRataplan

Copy link
Copy Markdown
Collaborator Author

Hm, something is going wrong with namespaces. Give me a bit of time to fix that

@DrRataplan

Copy link
Copy Markdown
Collaborator Author

Those work now. @JoernT this is ready to review!

@DrRataplan
DrRataplan requested a review from JoernT July 23, 2026 09:31

@rkottmann rkottmann left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

May I ask to remove the xlizenz naming from demo and in content?

@JoernT

JoernT commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

@rkottmann we'll do

@DrRataplan
DrRataplan force-pushed the create-nodes-child-elements branch from 9ad8a5e to 82eddfb Compare July 23, 2026 11:12
@DrRataplan DrRataplan changed the title fix(create-nodes): allow settng the values of elements in createnodes fix(create-nodes): allow setting the values of elements in createnodes Jul 28, 2026
@DrRataplan
DrRataplan force-pushed the create-nodes-child-elements branch from 82eddfb to 118a99b Compare July 28, 2026 08:57
And refactor a bit, process more as path steps, and see both attributes and elements as value tests

Fixes #362
@DrRataplan
DrRataplan force-pushed the create-nodes-child-elements branch from 118a99b to 3484991 Compare July 28, 2026 08:58
@JoernT
JoernT merged commit 1f41058 into dev Jul 29, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create nodes does not set the values of elements

3 participants