fix: handle sub folders in runbook rule setup form #1195
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
π Description
This pull request updates the logic for generating runbook folder options in the runbook rule form, allowing the UI to present all nested folder paths for each runbook file instead of just the top-level folder. This improves the granularity and flexibility of folder-based selections in the runbook rule setup.
π Type of Change
π Changes Made
extract-folderfunction is replaced withextract-all-folders, which extracts all possible folder paths from a runbook file name, supporting nested folders (e.g.,'postgres-demo/queries/file.runbook.sql'yields both'postgres-demo/'and'postgres-demo/queries/').generate-runbook-optionsis updated to use all extracted folders, ensuring that each unique folder path is presented as a selectable option, and preventing duplicate folder entries in the options list.π§ͺ Testing
Test Configuration:
Tests performed:
πΈ Screenshots
β Checklist