Skip to content

Conversation

@juliusknorr
Copy link
Collaborator

Implement transactions for dav methods. Since we kind of have different services in the root collection, i made them different transaction names based on the first path segment.

Also allows us to have profiles of dav requests

Copy link
Owner

@ChristophWurst ChristophWurst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice!

Signed-off-by: Julius Knorr <jus@bitgrid.net>
@juliusknorr
Copy link
Collaborator Author

Mind having another look? Trying to get my open PR count down ;)

Copy link
Owner

@ChristophWurst ChristophWurst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

Sorry for the delay. This drowned in my notifications.

@ChristophWurst ChristophWurst merged commit 4df9ca5 into master Jan 17, 2025
23 checks passed
@ChristophWurst ChristophWurst deleted the feat/dav branch January 17, 2025 11:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants