fix: duplicate folder path when selecting target folder for progress tracks - #640
Merged
zkat merged 1 commit intoJan 13, 2026
Merged
Conversation
…rogress track modal The "New progress track" modal was creating a duplicate FolderTextSuggest instance on the target folder input element. RelativeFolderSearchComponent already creates its own internal FolderTextSuggest in its constructor, so the extra instantiation on line 163 was redundant. This caused a bug where selecting a folder from the dropdown would result in the campaign root folder being duplicated in the path (e.g., "campaign1/campaign1/Progress" instead of "campaign1/Progress").
Contributor
Author
zkat
approved these changes
Jan 13, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
closes #639
Description
Problem
When creating a new progress track and selecting a folder from the "Target folder" dropdown, the campaign root folder was being duplicated in the resulting path.
For example, if the campaign folder is
campaign1and the user selectscampaign1/Progressfrom the dropdown, the modal would show:campaign1/Progresscampaign1/campaign1/ProgressThis resulted in entity blocks being created with incorrect underlying paths.
Root Cause
In
progress-create.ts, aFolderTextSuggestwas being explicitly instantiated on the target folder input element:However,
RelativeFolderSearchComponentalready creates its own internalFolderTextSuggestin its constructor. This created two competing suggest components on the same input:baseFolderbaseFolder(default)When a user selected a folder from the dropdown, the second suggest component would set the input value as a path relative to the vault root. Then
RelativeFolderSearchComponent.onChanged()would join this with the campaign folder, resulting in the duplicated path.Why This Only Affected Progress Tracks
The change is isolated to progress-create.ts only. Searching through the codebase, no other files seem to have the same duplicate pattern, so there's minimal risk of impacting anything else.
For example, the clock creation modal (
clock-create-modal.ts) does not have this extraFolderTextSuggestinstantiation. It correctly relies solely on the one provided byRelativeFolderSearchComponent. Similarly, other entity creation flows (like Roll All oracles) use different modal code that doesn't have this issue.Fix
Removed the redundant
FolderTextSuggestinstantiation and its now-unused import.Testing