Skip to content

Prevent invalid folder settings in the UI - #7063

Open
BartChris wants to merge 1 commit into
kitodo:mainfrom
BartChris:disable_invalid_folder_configs
Open

Prevent invalid folder settings in the UI#7063
BartChris wants to merge 1 commit into
kitodo:mainfrom
BartChris:disable_invalid_folder_configs

Conversation

@BartChris

@BartChris BartChris commented May 29, 2026

Copy link
Copy Markdown
Collaborator

This Pull Request adresses the UI related parts of the problems described in #6680 by preventing invalid values.

Two things are prevented:

a) A folder configured as source for image generation can not at the same time have its images generated:

image image

b) A folder which images are generated from another source cannot be setup to be used as a source for image generation.

image image

Configurations which were invalid before can still be opened, but if they are changed they can only be changed to a valid configuration.

@codacy-production

codacy-production Bot commented May 29, 2026

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 duplication

Metric Results
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@solth solth added the project management project and template related topics label Jun 1, 2026
@BartChris
BartChris force-pushed the disable_invalid_folder_configs branch 4 times, most recently from d1ec2e7 to 0823c52 Compare August 31, 2026 11:52
@BartChris
BartChris force-pushed the disable_invalid_folder_configs branch from 0823c52 to cb1ea94 Compare August 31, 2026 11:52
@BartChris
BartChris marked this pull request as ready for review August 31, 2026 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

project management project and template related topics

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants