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.

Sign up for GitHub

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

Jump to bottom

Apply WordPress filter to all configs #9

Closed

QWp6t wants to merge 1 commit into master from config-filter

Sponsor Member

QWp6t commented Mar 23, 2019

Feel free to thumbs up/down this idea.

Pros

provides a more accessible api for acorn newcomers
crude method to ensure a particular config value is applied

Cons

misuse of the filter (esp. by third parties) could result in difficult bugs to squash


          Apply WordPress filter to all configs

7d97aae

QWp6t closed this

QWp6t deleted the config-filter branch

July 8, 2019 14:15

Besmerch123 mentioned this pull request

After updating Illuminate packages to 8.37 version application is broked #102

Closed

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