-
Notifications
You must be signed in to change notification settings - Fork 5
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.
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
feat(braze): allowing braze content proxy to access lists #705
Conversation
Plan Result (braze-content-proxy-cdk-production)
|
584f28d
to
dd90f84
Compare
553a0c1
to
a5184c5
Compare
a5184c5
to
79ebbf1
Compare
79ebbf1
to
0c6486d
Compare
0c6486d
to
244e472
Compare
.filter((edge) => edge.node.item.__typename == 'Item') | ||
.map((edge) => { | ||
const item = edge.node.item; | ||
if (item.__typename == 'Item') { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is the typescript compiler not inferring this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no its not. and its very annoying
servers/braze-content-proxy/src/routes/scheduledItems.integration.ts
Outdated
Show resolved
Hide resolved
servers/braze-content-proxy/src/routes/scheduledItems.integration.ts
Outdated
Show resolved
Hide resolved
servers/braze-content-proxy/src/routes/scheduledItems.integration.ts
Outdated
Show resolved
Hide resolved
244e472
to
96b779f
Compare
🎉 This issue has been resolved in version @pocket-tools/terraform-modules-v5.14.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Goal
Enable future work for digest emails
Accomplishes
https://mozilla-hub.atlassian.net/browse/POCKET-10480