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

Adds option to utilize afterSave instead of afterCommit - cake 4.x #72

Merged
merged 5 commits into from
Jan 10, 2024

Conversation

cnizzardini
Copy link
Collaborator

@cnizzardini cnizzardini commented Jan 6, 2024

Adds option to utilize afterSave instead of afterCommit. This works better with transactions. To configure you set the following in your applications config such as config/app.php:

'AuditStash' => [
        'saveType' => 'afterSave',
]

@cnizzardini cnizzardini changed the title Adds option to utilize afterSave instead of afterCommit Adds option to utilize afterSave instead of afterCommit - cake 4.x Jan 6, 2024
@cnizzardini
Copy link
Collaborator Author

If there is no disagreement @dereuromark and @arusinowski I am going to merge this and do a 3.2 release to finally get all my projects off my fork.

@arusinowski
Copy link
Collaborator

Everything looks good, i see no issues

@cnizzardini cnizzardini merged commit b59dbb8 into lorenzo:3.x Jan 10, 2024
13 checks passed
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.

2 participants