Ping Protection V2#187
Conversation
SCDerox
left a comment
There was a problem hiding this comment.
Thanks so much for your contribution. Unfortunately, it seems like some changes occurred in the core that make your current version of this module unsupported.
Could you please run the config migration script to bring the existing configuration templates into support with the updated core requirements?
Please request a re-review once the PR is ready for review 😄
|
Merging the main branch actually fixed this lol, it's now back and ready for review ^^ |
|
Sorry about the 2 separate commits, I am on mobile doing them on the GitHub editor. I think this should be all now ^^ |
SCDerox
left a comment
There was a problem hiding this comment.
Just a few code quality issues 😄
|
Thanks for the feedback ^^ |
Ping Protection V2: QoL changes, multiple bug fixes, and a new user panel rewritten from scratch
Ping protection V2 was due for a bit of time, and it comes with multiple new (QoL) features/changes, multiple bug fixes and more. Due to the rewrite of the user panel being way more extensive, the lines of code have raised by quite the bit in some files (while some lowered). I've tried to keep the code as readable as possible in the meantime.
Full changelog:
And that's PP v2!
AI usage: