Skip to content

Add diagnostics tracking for BC5 google methods#876

Merged
tonidero merged 1 commit into
mainfrom
toniricodiez/sdk-2965-merge-main-changes-into-bc5-support
Apr 5, 2023
Merged

Add diagnostics tracking for BC5 google methods#876
tonidero merged 1 commit into
mainfrom
toniricodiez/sdk-2965-merge-main-changes-into-bc5-support

Conversation

@tonidero

Copy link
Copy Markdown
Contributor

Description

Depends on https://github.com/RevenueCat/khepri/pull/5488
It's based off #875
Completes SDK-2965

This adds diagnostics tracking for Google's methods in the BC5 branch.

@tonidero tonidero marked this pull request as ready for review March 15, 2023 20:11
@tonidero tonidero requested review from a team, beylmk and joshdholtz March 15, 2023 20:12
@codecov

codecov Bot commented Mar 15, 2023

Copy link
Copy Markdown

Codecov Report

Merging #876 (4873a47) into main (3407671) will decrease coverage by 2.51%.
The diff coverage is 92.85%.

❗ Current head 4873a47 differs from pull request most recent head 72be5b6. Consider uploading reports for the commit 72be5b6 to get more accurate results

@@            Coverage Diff             @@
##             main     #876      +/-   ##
==========================================
- Coverage   84.79%   82.28%   -2.51%     
==========================================
  Files         166      150      -16     
  Lines        5794     5381     -413     
  Branches      798      759      -39     
==========================================
- Hits         4913     4428     -485     
- Misses        563      671     +108     
+ Partials      318      282      -36     
Impacted Files Coverage Δ
.../com/revenuecat/purchases/google/BillingWrapper.kt 80.64% <91.89%> (+3.77%) ⬆️
...rchases/common/diagnostics/DiagnosticsEventName.kt 100.00% <100.00%> (ø)
...purchases/common/diagnostics/DiagnosticsTracker.kt 96.36% <100.00%> (+0.06%) ⬆️

... and 56 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@joshdholtz joshdholtz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

Base automatically changed from merge-main-bc5-support-2023-03-15 to bc5-support March 16, 2023 11:30
@tonidero tonidero force-pushed the toniricodiez/sdk-2965-merge-main-changes-into-bc5-support branch from 67e9fc7 to 4873a47 Compare March 16, 2023 11:38

@beylmk beylmk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🖲️

@tonidero

Copy link
Copy Markdown
Contributor Author

Thanks for the reviews! Holding until the backend part is merged.

@aboedo

aboedo commented Mar 27, 2023

Copy link
Copy Markdown
Member

@tonidero is the backend side merged already? if not, what's holding it back?

@tonidero

Copy link
Copy Markdown
Contributor Author

Not yet, just asked @thecadams for a review. Hopefully we can get that deployed soon

@tonidero tonidero mentioned this pull request Mar 30, 2023
2 tasks
@tonidero tonidero changed the base branch from bc5-support to main April 5, 2023 16:28
@tonidero tonidero force-pushed the toniricodiez/sdk-2965-merge-main-changes-into-bc5-support branch from 4873a47 to 72be5b6 Compare April 5, 2023 16:28
@tonidero

tonidero commented Apr 5, 2023

Copy link
Copy Markdown
Contributor Author

Backend changes were deployed and tested that these diagnostics are sent without errors. Merging now.

@tonidero tonidero enabled auto-merge (squash) April 5, 2023 16:38
@tonidero tonidero merged commit 70f4737 into main Apr 5, 2023
@tonidero tonidero deleted the toniricodiez/sdk-2965-merge-main-changes-into-bc5-support branch April 5, 2023 16:48
tonidero added a commit that referenced this pull request Apr 5, 2023
**This is an automatic release.**

### Bugfixes
* Remove flavors temporarily to fix deployment (#953) via Toni Rico
(@tonidero)
### Other Changes
* Add diagnostics tracking for BC5 google methods (#876) via Toni Rico
(@tonidero)

---------

Co-authored-by: revenuecat-ops <ops@revenuecat.com>
Co-authored-by: Toni Rico <antonio.rico.diez@revenuecat.com>
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