Skip to content

Conversation

@denessapi
Copy link
Collaborator

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Feb 14, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.92%. Comparing base (a0f74dc) to head (eb3a6c9).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #60      +/-   ##
==========================================
+ Coverage   96.79%   96.92%   +0.13%     
==========================================
  Files          16       16              
  Lines         717      749      +32     
  Branches      132      135       +3     
==========================================
+ Hits          694      726      +32     
  Misses         22       22              
  Partials        1        1              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@denessapi denessapi requested review from gbence and s4rd3d February 14, 2025 16:49
@denessapi denessapi force-pushed the denes-metadata-submission branch 3 times, most recently from 8a6fb1b to d0d58ea Compare February 18, 2025 09:26
Denes Sapi added 3 commits February 18, 2025 11:00
When the user is inactive for a longer period of time only the same
metadata events (and header events) were submitted periodically.
This change modifies the behaviour so, that if the user is inactive
for a longer period of time no data is submitted, and once the user
becomes active again a new metadata event is submitted.
@denessapi denessapi force-pushed the denes-metadata-submission branch from d0d58ea to eb3a6c9 Compare February 18, 2025 10:02
Copy link
Contributor

@gbence gbence left a comment

Choose a reason for hiding this comment

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

👍

@denessapi denessapi merged commit 9b8c3dd into main Feb 18, 2025
2 checks passed
@denessapi denessapi deleted the denes-metadata-submission branch February 18, 2025 11:47
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.

3 participants