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

Remove audit feature #8

Merged
merged 3 commits into from
May 29, 2024
Merged

Remove audit feature #8

merged 3 commits into from
May 29, 2024

Conversation

davidcoutadeur
Copy link

Closes #7

@davidcoutadeur davidcoutadeur added documentation Improvements or additions to documentation enhancement New feature or request labels May 17, 2024
@davidcoutadeur davidcoutadeur self-assigned this May 17, 2024
@coudot coudot changed the title 7 remove audit feature Remove audit feature May 28, 2024
@coudot
Copy link
Member

coudot commented May 28, 2024

Seems this PR includes the other (#6), you should remove the commit.

And a note should be added in upgrade process to inform that audit feature is removed (in favor of logback)

@davidcoutadeur davidcoutadeur force-pushed the 7-remove-audit-feature branch from db5bdc0 to ca90b1b Compare May 29, 2024 09:56
@davidcoutadeur
Copy link
Author

Seems this PR includes the other (#6), you should remove the commit.

I have rebased on top of master, should be ok now

And a note should be added in upgrade process to inform that audit feature is removed (in favor of logback)

Done in last commit.

@coudot coudot merged commit faaca8a into main May 29, 2024
@coudot coudot added this to the 2.2 milestone Aug 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove audit feature
2 participants