Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

have breadcrumbs in sample events use timestamp if it exists #3841

Merged
merged 1 commit into from Jul 29, 2016

Conversation

macqueen
Copy link
Contributor

@macqueen macqueen commented Jul 29, 2016

This was making percy fail:
https://percy.io/getsentry/sentry/builds/89438
https://percy.io/getsentry/sentry/builds/89403

@getsentry/infrastructure @benvinegar

This change is Reviewable

@macqueen macqueen changed the title wip - have breadcrumbs in sample events use timestamp if it exists have breadcrumbs in sample events use timestamp if it exists Jul 29, 2016
@benvinegar
Copy link
Contributor

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks broke.


Comments from Reviewable

@macqueen macqueen merged commit f879a32 into master Jul 29, 2016
@macqueen macqueen deleted the fix-percy-timestamps branch July 29, 2016 22:10
@github-actions github-actions bot locked and limited conversation to collaborators Dec 23, 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.

None yet

2 participants