Recently, we made two fixes to `SegmentConfig` to fix NPEs that were preventing some events from being sent to Segment. 1. https://github.com/appsmithorg/appsmith/pull/32498 2. https://github.com/appsmithorg/appsmith/pull/32351 But this ended up sending _too much_ to Segment now. That, and clearly we weren't missing anything by not sending those events with `null` in them. So we're bringing back that protection of NPEs. The protection of not sending value-less events to Segment. But in a more, educated way. We're adding `null` checks, and not sending anything when we see a `null`. Considering that context, a more accurate diff to review would be between the [`SegmentConfig` before those two PRs]( |
||
|---|---|---|
| .. | ||
| src | ||
| auth-notes | ||
| pom.xml | ||