Skip to content

v1.7.0: remove unreliable price data#17

Merged
ch-bas merged 1 commit into
mainfrom
feature/nextjs-frontend
Jun 18, 2026
Merged

v1.7.0: remove unreliable price data#17
ch-bas merged 1 commit into
mainfrom
feature/nextjs-frontend

Conversation

@ch-bas

@ch-bas ch-bas commented Jun 18, 2026

Copy link
Copy Markdown
Owner

Removed:

  • All MSRP price fields (msrp_usd + localized msrp_eur/gbp/inr/aed/aud/ cad/vnd/chf) from the schema and stripped from 468 cameras. The data was sparse (~36%, single-currency, undated) and unreliable — omitting it is more honest than publishing inaccurate prices.

Changed:

  • Clarified that video.streams[] describes stream capabilities, distinct from configs.frigate.* (the RTSP URLs to use) — they complement.

Docs: README schema table + roadmap updated; JSON-Schema-in-CI marked done.

What does this PR do?


Checklist

For new cameras

  • JSON file is under cameras/<brand-slug>/<model-slug>.json
  • id is a unique lowercase slug matching the file path
  • brand, model, type, resolution are all present
  • At least one URL in sources (manufacturer datasheet or reputable retailer)
  • npm run build passes locally with no errors

For corrections

  • Include a source URL confirming the correct value
  • npm run build passes locally with no errors

For schema / tooling changes

  • Existing cameras still validate (npm run build)
  • Updated docs/glossary.md if new fields were added

Source(s)


Notes

Removed:
- All MSRP price fields (msrp_usd + localized msrp_eur/gbp/inr/aed/aud/
  cad/vnd/chf) from the schema and stripped from 468 cameras. The data was
  sparse (~36%, single-currency, undated) and unreliable — omitting it is
  more honest than publishing inaccurate prices.

Changed:
- Clarified that video.streams[] describes stream capabilities, distinct
  from configs.frigate.* (the RTSP URLs to use) — they complement.

Docs: README schema table + roadmap updated; JSON-Schema-in-CI marked done.
@ch-bas

ch-bas commented Jun 18, 2026

Copy link
Copy Markdown
Owner Author

closes #16

@ch-bas ch-bas merged commit de63f2b into main Jun 18, 2026
2 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