[EP-238] CTA Clicked (discover_filter)

[moderateepheezy]

📲 What

Update discovery_category_name event property to be the name of the category if the parent field in DiscoveryParam of a selected filter is nil

🤔 Why

When selecting a category_name filter (e.g. Crafts as the category and then no subcategory selected, just "All Crafts Projects") from the discover overlay on the iOS app, this is what should be sent:

context_cta = discover_filter CORRECT

context_location = discover_overlay CORRECT

context_type = category_name CORRECT

discover_category_name = Crafts <<< MISSING

discover_subcategory_name = Crafts CORRECT

The issue is only when the user selects a category, but does not select a subcategory. In that situation, the property discover_category_name is missing.

👀 See

✅ Acceptance criteria

1. Select a Category in drop down menu on Discovery Tab. e.g Craft
2. Select All Craft Category
3. Verify on segment that

discover_category_name = "Craft"
discover_subcategory_name = "Craft"

[codecov]

Codecov Report

Merging #1447 (d5f142e) into master (7a7d0a6) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #1447   +/-   ##
=======================================
  Coverage   86.10%   86.11%           
=======================================
  Files        1107     1107           
  Lines       98727    98753   +26     
=======================================
+ Hits        85013    85038   +25     
- Misses      13714    13715    +1     

Impacted Files	Coverage Δ
Library/Tracking/KSRAnalytics.swift	85.46% <100.00%> (+0.01%)	⬆️
Library/Tracking/KSRAnalyticsTests.swift	100.00% <100.00%> (ø)
Library/Navigation.swift	76.89% <0.00%> (-0.37%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7a7d0a6...d5f142e. Read the comment docs.

[singhhari]

lgtm