Skip to content

Fix metadata config not written back to properties in application_con… - #55

Merged
bbpennel merged 2 commits into
UNC-Libraries:mainfrom
dtenenholtz:fix/deserializer-metadata-writeback
Aug 6, 2026
Merged

bbpennel merged 2 commits into
UNC-Libraries:mainfrom
dtenenholtz:fix/deserializer-metadata-writeback

Conversation

@dtenenholtz

Copy link
Copy Markdown
Contributor

When the 'metadata' config key is a plain string, make_paths_absolute rebuilds it as a hash but never assigns it back into properties, so the validator still sees the original string and raises 'path must not be empty'.

[Please note: this pull request and the other related one are both what Claude Code noticed and tried on its own to fix]

dtenenholtz and others added 2 commits July 24, 2026 12:11
…fig_deserializer

When the 'metadata' config key is a plain string, make_paths_absolute rebuilds
it as a hash but never assigns it back into properties, so the validator still
sees the original string and raises 'path must not be empty'.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@bbpennel
bbpennel merged commit bb094c9 into UNC-Libraries:main Aug 6, 2026
7 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