guardian / guardian/identity-processes

Log updates

Open
#31 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Scala
Stars
0
Forks
1
PR merge metrics
No merged PRs in 30d

Description

For the most part, there is no need to include as much information in the log entries - talk to @Mullefa for more details.

Also, it'd be useful to include the test variant when logging information about ab tests e.g. [here](https://github.com/guardian/identity-processes/blob/master/payment-failure/src/main/scala/com/gu/identity/paymentfailure/BrazeEmailService.scala#L92).

Contributor guide

No contributing guide indexed for this repository

Research direction

Talk to @Mullefa to clarify which log entries should contain less information and how the test variant should be represented. Start with the logging around BrazeEmailService.scala line 92, then identify the affected AB-test logging and confirm that the resulting entries contain the agreed information.

Written by the indexing model from the issue text.

Assessment

Tech stack
scala
Domain
observability
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.