zooniverse / talk-api

Apache License 2.0
6 stars 0 forks source link

Fix talk data exports to exclude empty boards #348

Closed yuenmichelle1 closed 1 month ago

yuenmichelle1 commented 1 month ago

See Slack Thread Here: https://zooniverse.slack.com/archives/C06DCM0V9/p1720800370052409 Tech details of issue found here: https://zooniverse.slack.com/archives/C06DCM0V9/p1720800400003809?thread_ts=1720800370.052409&cid=C06DCM0V9

This PR fixes the issue: that Talk data_exports fail when there is an empty board (i.e. a board without discussion or comment).

Change: Ensure the sql view created does not include empty boards so that when querying comment exports view, we ensure that each comment has a comment id (primary key of the sql view created).