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
Refactor of event insertion #5909
base: main
Are you sure you want to change the base?
Conversation
d4e9f87
to
05280ee
Compare
b826ae1
to
65be484
Compare
This PR is stale because it has been open 90 days with no activity. Remove stale label or comment or this will be closed in 5 days |
068bba7
to
a41a217
Compare
This PR is stale because it has been open 90 days with no activity. Remove stale label or comment or this will be closed in 5 days |
This PR is stale because it has been open 90 days with no activity. Remove stale label or comment or this will be closed in 5 days |
Automatically closed due to inactivity |
Spiking on automatic insertion of events, using
ApiKey
model as it's the simplest.Edit: Also
Attachment
andFileOperation
.Needs more type-safety before continuing down this path, but overall this will be a ton more reliable than the current manual hookup approach and remove some largely useless commands that only exist to wrap event creation in a transaction