Done-column "Archive all", coverage fix on repo switch, quieter console#74
Merged
Conversation
…sole Archive all (Done column): - New "Archive all" button in the Done column that archives every GitHub issue card in Done at once (after a confirm), reusing the existing per-card archive flow. Shown only when signed in and Done has issues; it lives in the column body so it hides with the collapsed column. Coverage card on repo switch: - fetchCoverageStatus now ignores its response if the user switched repos while the request was in flight, so a slow/empty result for the previous repo can't overwrite the current one. Switching away from dashban (to a repo with no coverage) and back was leaving coverage stuck on "unknown". Quieter console: - New src/logger.js (loaded first) gates console.log so the browser console stays clean by default; set window.DASHBAN_DEBUG = true to see the informational output. console.error / console.warn are untouched. Full suite green at 100% coverage; lint clean (976 tests). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_019VQjXJyM3EZPd4joy3EwaB
|
🚅 Deployed to the dashban-pr-74 environment in dashban
|
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
2 participants
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.
Three small board improvements in one PR.
1. "Archive all" button (Done column)
A new Archive all button in the Done column archives every GitHub issue card there at once (after a confirm), reusing the existing per-card archive flow (adds the
archivelabel, removes the card).2. Coverage card stuck on "unknown" after switching repos
Reported: add a second repo, switch to it, switch back to dashban → the coverage card shows "unknown".
Root cause was a stale-response race: a slow coverage fetch for the previous repo (which has no coverage) resolved after the switch back and overwrote dashban's value.
fetchCoverageStatusnow records the repo it was issued for and drops the result if the repo changed while the request was in flight (both success and error paths).3. Quieter browser console
The app logs a lot of routine lifecycle/context detail via
console.log(the📦/💾/📊lines). Newsrc/logger.js(loaded first) gatesconsole.logso the console stays clean by default.window.DASHBAN_DEBUG = true(from devtools) to see the informational output again.console.error/console.warnare untouched, so genuine problems still surface.console.log; the gate is applied once at load.Testing
npm test— 976 passing (added archive-all, stale-coverage, and logger tests).npm run test:coverage— 100% statements/branches/functions/lines across all files.npm run lint— 0 errors (7 pre-existing warnings unchanged).🤖 Generated with Claude Code
https://claude.ai/code/session_019VQjXJyM3EZPd4joy3EwaB
Generated by Claude Code