Summary
include_tags and include_description were designed in #88 but deferred because Watch does not yet have tags or description fields. Once those fields exist on Watch, add them to ContentOptions.
Blocked by
- Watch model gaining
tags (list of strings) and description (text) fields
Scope (once unblocked)
- Add
include_tags: bool = False and include_description: bool = False to ContentOptions in src/api/schemas/content_config.py
- Enrich
WatchEvent.metadata in _watch_base_metadata() (src/workers/tasks.py):
tags: watch.tags if set
description: watch.description if set
- Add
_build_tags_section() and _build_description_section() to src/core/notifications/content.py
- Add checkboxes to
notification_content_options.html
- Tests for new sections in
tests/core/notifications/test_content.py
Notes
Module docstring in src/api/schemas/content_config.py already notes this deferral.
Summary
include_tagsandinclude_descriptionwere designed in #88 but deferred becauseWatchdoes not yet havetagsordescriptionfields. Once those fields exist onWatch, add them toContentOptions.Blocked by
tags(list of strings) anddescription(text) fieldsScope (once unblocked)
include_tags: bool = Falseandinclude_description: bool = FalsetoContentOptionsinsrc/api/schemas/content_config.pyWatchEvent.metadatain_watch_base_metadata()(src/workers/tasks.py):tags:watch.tagsif setdescription:watch.descriptionif set_build_tags_section()and_build_description_section()tosrc/core/notifications/content.pynotification_content_options.htmltests/core/notifications/test_content.pyNotes
Module docstring in
src/api/schemas/content_config.pyalready notes this deferral.