guardian / mobile-notifications-content

Lambda that reads from the content api stream and triggers notifications for followable content
2 stars 0 forks source link

Explicitly match on atoms to reduce logging #73

Closed lindseydew closed 4 months ago

lindseydew commented 4 months ago

What does this change?

This PR includes Atom into the match from content returned from Capi so that we can log this separately and reduce the noise in the logs incurred by lots of modifications to the football euros

How to test

I've tested on CODE the lambda still runs ok.

I haven't been able to fully reproduce the issue on CODE, but I think we will be able to observe whether this fixes the issues ok in PROD easily enough

How can we measure success?

Have we considered potential risks?

Images

Accessibility