Skip to content

Conversation

jonathanmos
Copy link
Member

What does this PR do?

Connects the custom endpoints configuration property to the request builder for exposure batches.

Motivation

What inspired you to submit this pull request?

Additional Notes

Anything else we should know when reviewing?

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Make sure you discussed the feature or bugfix with the maintaining team in an Issue
  • Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

@jonathanmos jonathanmos changed the title FL-1115: Support custom endpoints for exposures [flags] FL-1115: Support custom endpoints for exposures Oct 15, 2025
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.71%. Comparing base (740290d) to head (7519fdf).

Additional details and impacted files
@@                     Coverage Diff                      @@
##           feature/feature-flagging    #2942      +/-   ##
============================================================
+ Coverage                     70.69%   70.71%   +0.02%     
============================================================
  Files                           834      834              
  Lines                         30378    30380       +2     
  Branches                       5131     5132       +1     
============================================================
+ Hits                          21474    21481       +7     
+ Misses                         7444     7439       -5     
  Partials                       1460     1460              
Files with missing lines Coverage Δ
...com/datadog/android/flags/internal/FlagsFeature.kt 87.50% <100.00%> (+0.32%) ⬆️
...roid/flags/internal/net/ExposuresRequestFactory.kt 100.00% <100.00%> (ø)

... and 38 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jonathanmos jonathanmos marked this pull request as ready for review October 15, 2025 12:55
@jonathanmos jonathanmos requested review from a team as code owners October 15, 2025 12:55
@jonathanmos jonathanmos requested a review from typotter October 16, 2025 07:35
Copy link

@typotter typotter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jonathanmos jonathanmos merged commit 410c494 into feature/feature-flagging Oct 16, 2025
26 checks passed
@jonathanmos jonathanmos deleted the jmoskovich/ffl-1115/custom-exposure-endpoint branch October 16, 2025 07:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants