forgejo_admin/pal-e-docs-app
1
0
Fork 0
Code Issues 4 Pull requests 1 Projects Releases Packages Wiki Activity Actions

fix: remove sprint note type references #49

Merged
forgejo_admin merged 1 commit from 48-remove-sprint-note-type-references into main 2026-03-24 17:07:56 +00:00
Conversation 1 Commits 1 Files changed 2 +2 -4
forgejo_admin commented 2026-03-23 19:30:07 +00:00
Owner
Copy link

Summary

Remove the sprint note type from the frontend. The sprint note type was removed from the platform as part of pal-e-docs#193; these are the two remaining frontend references.

Changes

  • src/lib/colors.ts -- removed sprint: '#2563eb' from TYPE_COLORS map
  • src/routes/notes/[slug]/edit/+page.svelte -- removed 'sprint' from NOTE_TYPES dropdown array, updated comment from 16 to 15

Test Plan

  • npm run build succeeds
  • No remaining sprint references in src/ (verified via grep)
  • Edit page dropdown no longer offers sprint as a note type

Review Checklist

  • Passed automated review-fix loop
  • No secrets committed
  • No unnecessary file changes
  • Commit messages are descriptive

Related

## Summary Remove the `sprint` note type from the frontend. The sprint note type was removed from the platform as part of pal-e-docs#193; these are the two remaining frontend references. ## Changes - `src/lib/colors.ts` -- removed `sprint: '#2563eb'` from TYPE_COLORS map - `src/routes/notes/[slug]/edit/+page.svelte` -- removed `'sprint'` from NOTE_TYPES dropdown array, updated comment from 16 to 15 ## Test Plan - [x] `npm run build` succeeds - [x] No remaining `sprint` references in `src/` (verified via grep) - [x] Edit page dropdown no longer offers sprint as a note type ## Review Checklist - [x] Passed automated review-fix loop - [x] No secrets committed - [x] No unnecessary file changes - [x] Commit messages are descriptive ## Related - Closes forgejo_admin/pal-e-app#48 -- the Forgejo issue this PR addresses - forgejo_admin/pal-e-docs#193 -- parent cleanup issue removing sprint note type from the platform
fix: remove sprint note type references
Some checks failed
ci/woodpecker/pr/woodpecker Pipeline failed
Details
602f7589b4 
Sprint note type has been removed from the platform (pal-e-docs#193).
Clean up the two remaining frontend references: color map and edit
dropdown.

Closes #48

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
forgejo_admin commented 2026-03-23 19:30:28 +00:00
Author
Owner
Copy link

Review: LGTM

Files reviewed: 2/2

Findings

No issues found. This is a clean two-line removal:

  1. src/lib/colors.ts -- sprint: '#2563eb' removed from TYPE_COLORS. Trailing comma correctly adjusted on the now-final incident entry.
  2. src/routes/notes/[slug]/edit/+page.svelte -- 'sprint' removed from NOTE_TYPES array. Comment count updated from 16 to 15.

Verification

  • npm run build passes
  • grep -r sprint src/ returns zero matches
  • No unrelated changes in diff
## Review: LGTM **Files reviewed:** 2/2 ### Findings No issues found. This is a clean two-line removal: 1. **`src/lib/colors.ts`** -- `sprint: '#2563eb'` removed from TYPE_COLORS. Trailing comma correctly adjusted on the now-final `incident` entry. 2. **`src/routes/notes/[slug]/edit/+page.svelte`** -- `'sprint'` removed from NOTE_TYPES array. Comment count updated from 16 to 15. ### Verification - `npm run build` passes - `grep -r sprint src/` returns zero matches - No unrelated changes in diff
forgejo_admin deleted branch 48-remove-sprint-note-type-references 2026-03-24 17:07:56 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
domain:backend

   
domain:devops

   
domain:frontend
No labels
domain:backend
domain:devops
domain:frontend
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
forgejo_admin/pal-e-docs-app!49
No description provided.