search

mewebstudio / Purifier

HTMLPurifier for Laravel 5/6/7/8/9/10/11
MIT License
1.87k stars 230 forks source link

laravel-10 support and fix required version format #186

Closed txdFabio closed 1 year ago

txdFabio commented 1 year ago

fixed compose require version format and addedd laravel-10 support

txdFabio commented 1 year ago

phpunit should also include 10: "phpunit/phpunit": "^8.0|^9.0|^10",

done!

faytekin commented 1 year ago

@mewebstudio ping

fabiofdsantos commented 1 year ago

ping @mewebstudio

digitv commented 1 year ago

@mewebstudio ping

tabacitu commented 1 year ago

Thanks for the PR @txdFabio 🙏 With your PR, Purifier::clean() function works just as fine in our L10 tests!

@mewebstudio when can we expect a merge & tag?

kminek commented 1 year ago

@mewebstudio are you okay? i noticed that you're from Turkey and there were some serious earthquakes there...

pxpm commented 1 year ago

@mewebstudio are you okay? i noticed that you're from Turkey and there were some serious earthquakes there...

I don't think the person is living in the affected area of the earthquake. If they didn't go to that area during the earthquake, they shouldn't have been affected.

I think that's a very rude way of putting up the issue. People have family, friends and empathy with other human beings.

There are many ways to get affected by the events, even if living in another country.

I make @kminek my words, and hope @mewebstudio , his relatives and friends are all ok 🙏

Also I am here with all other developers in the hopes that we can help @mewebstudio to merge this PR 👍

Cheers

ali-alharthi commented 1 year ago

If you're in a hurry, you can always find other packages or fork this one. For example the following stevebauman/purify. Good luck.

eduardopilati commented 1 year ago

If you're in a hurry, you can always find other packages or fork this one. For example the following stevebauman/purify. Good luck.

or making your own fork

txdFabio commented 1 year ago

I've seen you have merged the PR, thanks. Can you also tag for a new release, please?