Settings: the ignore list keeps the same app twice in different case
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 88/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- electron, typescript
- Ambito
- desktop
Direzione di ricerca
Inizia in src/shared/settings.ts, in corrispondenza di sanitizeSettings, poi esamina i test correlati di sanitizeSettings e isIgnoredApp in tests/settings-color.test.ts. Aggiungi un test con nomi di app che differiscono solo per maiuscole e minuscole e verifica che siano mantenuti la prima grafia e l’ordine dell’elenco. Il lavoro è concluso quando il test e pnpm test hanno esito positivo.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
The ignore list matches app names without regard to case, but adding an app doesn't check case. Adding 1password while 1Password is on the list adds a second chip:
sanitizeSettings({ ignoredApps: [...DEFAULT_IGNORED_APPS, '1password'] }).ignoredApps
// [..., 'Proton Pass', 'Passwords', '1password']
It removes exact duplicates and extra spaces already (' Bitwarden ' disappears), just not ones that differ in case.
Where to look
src/shared/settings.ts:sanitizeSettings, wherenew Set(...)removes duplicates. Both the settings file and every change from the Settings window go through ittests/settings-color.test.ts: thesanitizeSettingsandisIgnoredApptests
Done when
- Names that differ only in case are kept once, with the spelling that came first
- The order of the list is otherwise unchanged
- A test covers it, and
pnpm testpasses
Adding an app that's already on the list then just clears the field in Settings, which is fine.
Suggested pull request title: fix(settings): keep each ignored app once, whatever its case
New to OpenPaste? CONTRIBUTING.md covers setup, the code conventions and the checks to run before a pull request (pnpm typecheck, pnpm test, pnpm format, pnpm cleaner). Leave a comment if you're picking this up so nobody doubles up.
Generated by Claude Code
- Lingua principale
- TypeScript
- Stelle
- 0
- Fork
- 3
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di alexsilver8/openpaste
-
enhancement good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 83/100
alexsilver8/openpaste#31 ·
-
enhancement good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 92/100
alexsilver8/openpaste#32 ·
-
bug good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
alexsilver8/openpaste#30 ·
-
bug good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
alexsilver8/openpaste#29 ·
-
bug good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
alexsilver8/openpaste#27 ·
Tutte le issue di alexsilver8/openpaste
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
I maintainer di solito rispondono entro 1 giorno
-
high priority
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
CI-CMG/deep-sea-corals-exb#84 · 1 commento ·
-
Create/import wallet form shows raw "password must be at least 1 characters" instead of the translated messageForse già presa @CapThunder19 l’ha presa oggi. Apertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
joinmarket-webui/jam#1538 ·
I maintainer di solito rispondono entro 1 giorno
-
t0ggles
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
nolimits4web/swiper#8240 · 1 commento ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
I maintainer di solito rispondono entro 1 giorno