search

joomla-extensions / patchtester

Patch testing extension for the Joomla! CMS
https://issues.joomla.org
GNU General Public License v2.0
48 stars 44 forks source link

Drop jQuery, fix a number of XSS #364

Closed dgrammatiko closed 1 year ago

dgrammatiko commented 1 year ago

Pull Request for Issue # .

Summary of Changes

Testing Instructions

dgrammatiko commented 1 year ago

ping @roland-d

roland-d commented 1 year ago

@dgrammatiko Awesome, it is on my todo list but no time yet. Will review later.

dgrammatiko commented 1 year ago

fwiw I just converted the code, haven't tested it 😅

roland-d commented 1 year ago

@dgrammatiko That is fine, that is why I did not merge it for this release :P Will review and test.

roland-d commented 1 year ago

@heelc29 I am currently reviewing/updating this PR.

roland-d commented 1 year ago

@dgrammatiko I have tested this and there were quite a few things not working but all should be fixed now. Can you give it a check to see if there is nothing obvious wrong. Thanks.

Indexing the pull requests works now :)

dgrammatiko commented 1 year ago

LGTM, sorry for the silly mistakes 😀