Skip to content

feat(#16): add backers_count field to Campaign model#23

Merged
Jing-yilin merged 2 commits intodevelopfrom
feature/16-backers-count
Feb 27, 2026
Merged

feat(#16): add backers_count field to Campaign model#23
Jing-yilin merged 2 commits intodevelopfrom
feature/16-backers-count

Conversation

@Jing-yilin
Copy link
Contributor

Closes #16

Changes

  • Add BackersCount int to model.go
  • Populate from data-project JSON attribute in kickstarter_parser.go
  • Include in nightly cron upsert column list

Stack

PR 5/6. Base: #22 (parse alerting). Next: #18 (backfill)

Copy link
Contributor Author

@Jing-yilin Jing-yilin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One gap here: this only populates BackersCount on the HTML parser path used by cron. Search() still prefers AI extraction, and parseAIResponse() parses backers_count but never copies it onto the returned model.Campaign, so /api/campaigns and /api/campaigns/search responses will keep dropping the field whenever AI extraction succeeds. If the model now exposes backers_count, that path needs to be wired up too.

@Jing-yilin Jing-yilin force-pushed the feature/16-backers-count branch from a345c2a to 70b986a Compare February 27, 2026 10:26
- Add BackersCount int to model.go (gorm default:0)
- Populate from data-project JSON in kickstarter_parser.go
- Include in nightly cron upsert column list
@Jing-yilin Jing-yilin force-pushed the feature/15-parse-alerting branch from 2922605 to 7721938 Compare February 27, 2026 10:28
@Jing-yilin Jing-yilin force-pushed the feature/16-backers-count branch from 70b986a to 6bff708 Compare February 27, 2026 10:29
@Jing-yilin Jing-yilin changed the base branch from feature/15-parse-alerting to develop February 27, 2026 10:29
@Jing-yilin Jing-yilin merged commit 56b846b into develop Feb 27, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant