CaptainFact / CaptainFact/captain-fact
Don't allow null speakers on statements
- Dominant language
- No language data
- Stars
- 71
- Forks
- 7
- PR merge metrics
- No merged PRs in 30d
Description
- All new statements should be added with a speaker, and old statements without one should be migrated.
- Constraint should not be enforced in DB in a first time as we have existing statements with nil speakers.
- Frontend can be implemented first, independently of backend.
---
:information_source: Some work was started on the frontend by @julien-leclercq [here](https://github.com/CaptainFact/captain-fact-frontend/pull/176) but PR was dropped after being staled for too long in a buggy state. Can be used as reference, but conflicts now prevent the code to be re-used.
Contributor guide
Research direction
Review the issue requirements and the abandoned captain-fact-frontend PR #176 first, treating it as reference only because the issue says it is buggy and conflicts prevent reuse. Trace statement creation, existing nil-speaker records, and the frontend statement flow. Done means new statements have speakers, old records are migrated, and the database constraint remains deferred initially.
Written by the indexing model from the issue text.
Assessment
- Domain
- database, full-stack
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 25/100