Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Sync newsletter subscribed flag to Acoustic relational table #562

Closed
3 tasks
grahamalama opened this issue Feb 23, 2023 · 0 comments · Fixed by #752
Closed
3 tasks

Sync newsletter subscribed flag to Acoustic relational table #562

grahamalama opened this issue Feb 23, 2023 · 0 comments · Fixed by #752
Assignees
@leplatrem

Comments

@grahamalama
Copy link
Contributor

In Acoustic, there is a rt_moco_newsletters relational table that holds metadata related to newsletter subscriptions. It essentially mirrors the newsletters table in CTMS, except for a boolean subscribed field. This data lives in the main contact database as sub_<newsletter> columns.

We want to stop adding new columns to the main table, so we propose that we sync the newsletter.subscribed CTMS column to the Acoustic relational table.

  • add subscribed bool field to rt_moco_newsletters table
  • code change to sync newsletter.subscribed column to relational table
  • document new process for onboarding new newsletters, querying relational table
@grahamalama grahamalama mentioned this issue Feb 23, 2023
Closed
3 tasks
@leplatrem leplatrem mentioned this issue Jun 19, 2023
Merged
3 tasks
@leplatrem leplatrem self-assigned this Jul 7, 2023
leplatrem added a commit that referenced this issue Jul 7, 2023
@leplatrem
Fix #562: add subscribed field to newsletters Acoustic relational table
b19c0c3
@leplatrem leplatrem mentioned this issue Jul 7, 2023
Merged
3 tasks
leplatrem added a commit that referenced this issue Jul 25, 2023
@leplatrem
Fix #562: add subscribed field to newsletters Acoustic relational tab…
0666421 
…le (#752)

* Fix #562: add subscribed field to newsletters Acoustic relational table

* Improve tests

* Use newsletter lang and format in acoustic relational table
leplatrem added a commit that referenced this issue Jul 25, 2023
@leplatrem
Follow-up #562: fix bool column conversion
ffdca65
leplatrem added a commit that referenced this issue Jul 25, 2023
@leplatrem
Follow-up #562: fix bool column conversion (#777)
d44854f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@leplatrem leplatrem

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #562: add subscribed field to newsletters Acoustic relational table mozilla-it/ctms-api
2 participants
@leplatrem @grahamalama