Skip to content

Include name in orders export#5721

Merged
ab9rf merged 2 commits into
DFHack:developfrom
pajawojciech:orders-export-name
Jul 11, 2026
Merged

Include name in orders export#5721
ab9rf merged 2 commits into
DFHack:developfrom
pajawojciech:orders-export-name

Conversation

@pajawojciech

Copy link
Copy Markdown
Contributor

Exported orders now include a human-readable name field

obraz

Related issue: #4452

@ab9rf

ab9rf commented Feb 24, 2026

Copy link
Copy Markdown
Member

This PR is causing test "import_export_reaction_condition" to fail. This needs to be addressed before it can be considered for merging.

See test/plugins/orders.lua for the description of this test.

@pajawojciech

Copy link
Copy Markdown
Contributor Author

This PR is causing test "import_export_reaction_condition" to fail. This needs to be addressed before it can be considered for merging.

See test/plugins/orders.lua for the description of this test.

Fixed

@ab9rf

ab9rf commented Jul 11, 2026

Copy link
Copy Markdown
Member

Closes #4452

@ab9rf ab9rf merged commit a2220ff into DFHack:develop Jul 11, 2026
14 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.

2 participants