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

Crash if setting has no value #5 #6

Closed
wants to merge 1 commit into from
Closed

Conversation

mikeaix
Copy link

@mikeaix mikeaix commented Mar 14, 2023

Check if the setting has a value. If not, make it an empty string.

Check if the setting has a value. If not, make it an empty string.
@ByteDev
Copy link
Owner

ByteDev commented Mar 16, 2023

Added your fix to my master branch manually. For some reason GH wont let me merge your PR (button doesnt work work hmmm).

I've also published this fix in version 7.2.1 of the package.

Thanks!

@ByteDev ByteDev closed this Mar 16, 2023
@mikeaix
Copy link
Author

mikeaix commented Mar 16, 2023

Thank you for responding and integrating my changes so quickly! Especially with merging not working.

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