Skip to content
This repository has been archived by the owner on Aug 30, 2023. It is now read-only.

fix: update last session timestamp #437

Merged
merged 1 commit into from
May 29, 2020
Merged

fix: update last session timestamp #437

merged 1 commit into from
May 29, 2020

Conversation

marandaneto
Copy link
Contributor

@marandaneto marandaneto commented May 29, 2020

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

fix: update last session timestamp

💡 Motivation and Context

when app goes to foreground and get lastUpdatedSession, it should do the same on when app goes to background, so app won't stop and start sessions every time after the interval period.

💚 How did you test it?

running it

📝 Checklist

  • I reviewed submitted code
  • I added tests to verify changes
  • All tests passing

🔮 Next steps

@bruno-garcia
Copy link
Member

Good catch!

@codecov-commenter
Copy link

Codecov Report

Merging #437 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #437   +/-   ##
=========================================
  Coverage     60.26%   60.26%           
  Complexity      816      816           
=========================================
  Files            93       93           
  Lines          3727     3727           
  Branches        359      359           
=========================================
  Hits           2246     2246           
  Misses         1328     1328           
  Partials        153      153           

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 3d4a59e...ba33960. Read the comment docs.

@marandaneto marandaneto merged commit fb250f7 into getsentry:master May 29, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants