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

Fixing bug: Upsetting instead of inserting on `set`. #3

Merged

nathanb merged 4 commits into main from whoops-set-upsert

Nov 28, 2023

Member

nathanb commented Nov 28, 2023

Dog-fooding this, I had a session set happen, and I discovered a duplicate key conflict.

Set needs to be upsert instead of insert.
Added a test to ensure most of it.

nathanb added 4 commits

November 27, 2023 19:23


          fixing this bug.

8e984b7

wip

1eb641a


          I really need tests around this.

9f7188f


          fixing bug

33822ee

nathanb merged commit b302fdc into main

2 checks passed

nathanb deleted the whoops-set-upsert branch

November 28, 2023 01:48

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet