Skip to content

Add FR_IGNORE_NAMES (#91)#92

Closed
nikp123 wants to merge 1 commit intoerror311:masterfrom
nikp123:master
Closed

Add FR_IGNORE_NAMES (#91)#92
nikp123 wants to merge 1 commit intoerror311:masterfrom
nikp123:master

Conversation

@nikp123
Copy link
Copy Markdown
Contributor

@nikp123 nikp123 commented Jan 18, 2026

Adding this should enable users to specify what directories to ignore so that their indexing doesn't take too long or consume too much resources or possibly to hide even more metadata files (ie. .DS_store)

I haven't been able to test this due to the fact that I couldn't get the CI to run. Docker is giving me trouble, but I assume it should be correct because its quite literally the same method you used for FR_TRUSTED_PROXIES.

Anyway please test if you can and merge it it seems right.

Update: This doesn't seem to work. I don't know why.

error311 added a commit that referenced this pull request Jan 20, 2026
…UX polish (fixes #91, refs #92)

- add ignoreRegex setting (admin config) with env override FR_IGNORE_REGEX to hide folders from tree/counts/indexing
- add snapshot preset helper for common NAS snapshot paths (fixes #91)
- unify ignore logic via FS::shouldIgnoreEntry across folder counts, tree listing, and disk usage scans
- admin: improve settings search UX (clear button) + smoother section header styling
- UI: polish header dock collapse/expand icon animations (landing/lift + reduced-motion support)

Fixes #91
Refs #92

Co-authored-by: nikp123 <nikp123@e.email>
@error311
Copy link
Copy Markdown
Owner

Thanks for the PR! I implemented a working version of this idea in 96e15b5 while keeping your config naming. Credit added in the commit message. Closing this PR in favor of the merged implementation.

@error311 error311 closed this Jan 20, 2026
@nikp123
Copy link
Copy Markdown
Contributor Author

nikp123 commented Jan 20, 2026

That's fine. Thanks for your support

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