Add tags to atoms#211
Conversation
waisingyiu
force-pushed
the
ws/add-tag-to-atom
branch
from
cedf9c0 to
deb1ffb
Compare
SHession
left a comment
SHession
left a comment
There was a problem hiding this comment.
Code changes look good, it would be good to update the description with our revised approach for prosperity.
Thanks @SHession . I've updated the description. Great if you can take another look to see if it's accurate. |
…by waisingyiu waisingyiu published release version 11.0.0-PREVIEW.wsadd-tag-to-atom.2026-04-30T1425.deb1ffbf using gha-scala-library-release-workflow: https://github.com/guardian/gha-scala-library-release-workflow Release-Version: 11.0.0-PREVIEW.wsadd-tag-to-atom.2026-04-30T1425.deb1ffbf Release-Initiated-By: https://github.com/waisingyiu Release-Workflow-Run: https://github.com/guardian/content-atom/actions/runs/25170902764 Release-Notes: #211
|
@waisingyiu has published a preview version of this PR with release workflow run #49, based on commit deb1ffb: 11.0.0-PREVIEW.wsadd-tag-to-atom.2026-04-30T1425.deb1ffbf Want to make another preview release?Click 'Run workflow' in the GitHub UI, specifying the ws/add-tag-to-atom branch, or use the GitHub CLI command: gh workflow run release.yml --ref ws/add-tag-to-atom Want to make a full release after this PR is merged?Click 'Run workflow' in the GitHub UI, leaving the branch as the default, or use the GitHub CLI command: gh workflow run release.yml |
3 participants
This pull request adds a new property for a list of tag IDs to atoms as part of our work on making atoms taggable. The motivation is to improve data analysis using tracking data from data lake by splitting data based on the tags associated with the atom.
For this purpose, we just need the tag IDs on the atom. To avoid increasing the complexity of the work unnecessarily (particularly as this thrift model is shared between the tools and the CAPI Concierge), we simply add a list of tag IDs as strings to the atom model.