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.

Sign up for GitHub

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

Jump to bottom

Attach a uuid to each event so dedupe can happen server-side #16

Merged
merged 32 commits into from
Apr 21, 2024
Merged

Attach a uuid to each event so dedupe can happen server-side #16

merged 32 commits into from
Apr 21, 2024

Conversation

grahamc
Copy link
Member

@grahamc grahamc commented Apr 19, 2024

Description
Checklist
  • Tested changes against a test repository
  • Added or updated relevant documentation (leave unchecked if not applicable)
  • (If this PR is for a release) Updated README to point to the new tag (leave unchecked if not applicable)

@grahamc grahamc requested a review from lucperkins April 19, 2024 02:39
@netlify Netlify
Copy link

netlify bot commented Apr 19, 2024 •
edited
Loading

✅ Deploy Preview for detsys-ts-docs ready!

Name Link
🔨 Latest commit 6108a5c
🔍 Latest deploy log https://app.netlify.com/sites/detsys-ts-docs/deploys/66245c351f0bcd0009d88a1b
😎 Deploy Preview https://deploy-preview-16--detsys-ts-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@grahamc grahamc enabled auto-merge April 19, 2024 20:44
lucperkins
lucperkins requested changes Apr 19, 2024
View reviewed changes
package.json Outdated Show resolved Hide resolved
cole-h
cole-h reviewed Apr 19, 2024
View reviewed changes
.gitignore Outdated Show resolved Hide resolved
@lucperkins lucperkins mentioned this pull request Apr 20, 2024
Merged
grahamc
grahamc commented Apr 20, 2024
View reviewed changes
src/correlation.ts Outdated Show resolved Hide resolved
@lucperkins lucperkins merged commit ea7ff7d into main Apr 21, 2024
5 checks passed
@lucperkins lucperkins deleted the uuid-cleanup branch April 21, 2024 00:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cole-h cole-h cole-h left review comments

@lucperkins lucperkins lucperkins approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@grahamc @lucperkins @cole-h