Skip to content

Move Log Directory - #166

Open
n7studios wants to merge 2 commits into
mainfrom
move-log-directory
Open

Move Log Directory#166
n7studios wants to merge 2 commits into
mainfrom
move-log-directory

Conversation

@n7studios

Copy link
Copy Markdown
Contributor

Summary

Implements the Kit WordPress Libraries that change the log folder location from the Plugin's log folder to wp-content/uploads/kit-logs, ensuring Plugin checksum verification passes (Linear)

Deletes the log file on Plugin uninstallation, now the file lives outside of the Plugin folder.

Testing

Existing tests pass.

Checklist

@n7studios n7studios added the enhancement New feature or request label Aug 25, 2026
@n7studios n7studios self-assigned this Aug 25, 2026
@n7studios
n7studios requested review from a team, ciccio-kit and noelherrick and removed request for a team August 25, 2026 12:17
@n7studios
n7studios marked this pull request as ready for review August 25, 2026 12:17

@noelherrick noelherrick left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Same comment as in the libraries PR - should we move the log files vs. just a plain delete? I'm also wondering if we should have an API for the "uninstall" since this is probably something we want for every plugin

@n7studios

Copy link
Copy Markdown
Contributor Author

Same comment as in the libraries PR - should we move the log files vs. just a plain delete? I'm also wondering if we should have an API for the "uninstall" since this is probably something we want for every plugin

Answered here.

@n7studios
n7studios requested a review from noelherrick August 26, 2026 02:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants