bcgov / bcgov/SIMS

Cleanup - COE Declined

Open
#5,560 0 comments 0 reactions 0 assignees View on GitHub
Business Full-time Institution Part-time
Dominant language
TypeScript
Stars
29
Forks
15
PR merge metrics
No merged PRs in 30d

Description

Created based on feedback from #5446

**Acceptance Criteria**
- [ ] The reasons should all include a period ".". Update the tracker card to expect the input has a period and update the db reasons.
- [ ] Update/refactor API to validate reason ID on declined COE (excluding ECE as it's always "Other")
- [ ] Consider if changes in the display order are needed. (Currently ID desc)

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reading the feedback in issue #5446, then locate the tracker card, the declined-COE API, and the database reasons referenced here. Determine how reason IDs and ECE are currently handled, and decide whether display ordering needs adjustment. Done means the acceptance criteria are addressed and the related data and validation behavior are covered.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
api, backend, database
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.