Skip to content

[16.0] auditlog: Log exports - #2768

Merged
OCA-git-bot merged 1 commit into
OCA:16.0from
tarteo:16-auditlog-export-data
Sep 17, 2025
Merged

OCA-git-bot merged 1 commit into
OCA:16.0from
tarteo:16-auditlog-export-data

Conversation

@tarteo

@tarteo tarteo commented Nov 28, 2023

Copy link
Copy Markdown
Member

image

@tarteo
tarteo force-pushed the 16-auditlog-export-data branch from 7bed3a7 to b78db75 Compare November 28, 2023 14:07
@thomaspaulb

Copy link
Copy Markdown
Contributor

@tarteo Could you rebase?

@tarteo
tarteo force-pushed the 16-auditlog-export-data branch from b78db75 to 6382e4a Compare February 2, 2024 12:40
@tarteo

tarteo commented Feb 2, 2024

Copy link
Copy Markdown
Member Author

@tarteo Could you rebase?

Done!

Comment thread auditlog/views/auditlog_view.xml
Comment thread auditlog/views/auditlog_view.xml Outdated
Comment thread auditlog/models/rule.py Outdated
Comment thread auditlog/models/rule.py Outdated
Comment thread auditlog/models/rule.py Outdated
Comment thread auditlog/models/log.py
Comment thread auditlog/models/log.py
Comment thread auditlog/models/rule.py

@AungKoKoLin1997 AungKoKoLin1997 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Functional Review: LGTM

@yostashiro yostashiro left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice improvement.

Comment thread auditlog/models/rule.py Outdated
@tarteo
tarteo force-pushed the 16-auditlog-export-data branch 2 times, most recently from 9968169 to 0213b44 Compare August 6, 2025 08:58
@tarteo

tarteo commented Aug 6, 2025

Copy link
Copy Markdown
Member Author

Wow, the time flies by. I come here again requesting your review 😬 @yostashiro @thomaspaulb
Error seems to be unrelated to this: https://github.com/OCA/server-tools/actions/runs/16772374931/job/47490378259?pr=2768#step:8:4924

@StefanRijnhart

Copy link
Copy Markdown
Member

@thomaspaulb Is this one alright for you now?

@tarteo Needs another rebase because of conflicts.

@thomaspaulb

Copy link
Copy Markdown
Contributor

I'm good, so let's rebase and merge.

[IMP] Apply suggestions
@tarteo
tarteo force-pushed the 16-auditlog-export-data branch from 0213b44 to e2fb9f3 Compare September 17, 2025 13:22
@tarteo

tarteo commented Sep 17, 2025

Copy link
Copy Markdown
Member Author

@StefanRijnhart Done!

@StefanRijnhart

Copy link
Copy Markdown
Member

/ocabot merge major

@OCA-git-bot

Copy link
Copy Markdown
Contributor

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-2768-by-StefanRijnhart-bump-major, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit 7c9a86a into OCA:16.0 Sep 17, 2025
9 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at d2493d3. Thanks a lot for contributing to OCA. ❤️

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants