Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Drop jQuery, fix a number of XSS #364

Merged
merged 3 commits into from
Sep 6, 2023
Merged

Drop jQuery, fix a number of XSS #364

merged 3 commits into from
Sep 6, 2023

Conversation

dgrammatiko
Copy link

Pull Request for Issue # .

Summary of Changes

Testing Instructions

@dgrammatiko dgrammatiko marked this pull request as draft August 26, 2023 09:43
@dgrammatiko
Copy link
Author

ping @roland-d

@roland-d
Copy link
Collaborator

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

@dgrammatiko dgrammatiko marked this pull request as ready for review August 26, 2023 10:11
@dgrammatiko
Copy link
Author

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

@roland-d
Copy link
Collaborator

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

@roland-d
Copy link
Collaborator

@heelc29 I am currently reviewing/updating this PR.

dgrammatiko and others added 2 commits August 27, 2023 17:04
Co-authored-by: heelc29 <66922325+heelc29@users.noreply.github.com>
Signed-off-by: Roland Dalmulder <contact@rolandd.com>
@roland-d
Copy link
Collaborator

roland-d commented Sep 6, 2023

@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
Copy link
Author

LGTM, sorry for the silly mistakes 😀

@roland-d roland-d merged commit 6501996 into joomla-extensions:master Sep 6, 2023
@dgrammatiko dgrammatiko deleted the patch-1 branch September 6, 2023 20:39
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants