8000 Add: Retrieve log directory with getConfig() by mpconley · Pull Request #7877 · Mudlet/Mudlet · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add: Retrieve log directory with getConfig() #7877

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

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

mpconley
Copy link
Contributor
@mpconley mpconley commented May 31, 2025

Brief overview of PR changes/additions

Added support for getConfig("logDirectory") in the Lua API. Scripts can now retrieve the current profile's log directory path as a string.

Motivation for adding to Mudlet

Previously, there was no way for Lua scripts or packages to programmatically discover where log files are stored. This addition enables automation and scripting scenarios that need to access or display the log directory, as requested in #7823.

Other info (issues closed, discussion etc)

Closes #7823.
No impact on existing functionality; other getConfig options are unaffected.
Tested to ensure the correct directory is returned for the active profile.

Screenshot 2025-05-31 at 9 32 53 AM

@mpconley mpconley requested a review from a team as a code owner May 31, 2025 13:41
@add-deployment-links
Copy link
add-deployment-links bot commented May 31, 2025

Hey there! Thanks for helping Mudlet improve. 🌟

Test versions

You can directly test the changes here:

@mpconley mpconley self-assigned this May 31, 2025
@mpconley mpconley added the needs documentation This pull request changes things the players would use/see and thus needs an update in the manual label May 31, 2025
@mpconley
Copy link
Contributor Author

Brushed up https://wiki.mudlet.org/w/Manual:Miscellaneous_Functions#getConfig, it was missing some other options from the code that are now documented.

@mpconley mpconley removed the needs documentation This pull request changes things the players would use/see and thus needs an update in the manual label May 31, 2025
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.

Add log location to getConfig()
2 participants
0