Skip to content

Remove automod/MPV #400

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.

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

Open
wants to merge 2 commits into
base: 3.3.x
Choose a base branch
from
Open

Conversation

Skouat
Copy link
Contributor

@Skouat Skouat commented May 31, 2025

This PR handles the issue #365

@iMattPro
Copy link
Member

This makes me think we should also remove "Zend" as a search engine, then that would allow us to delete the zend and ezcomponents junk in includes/library

@Skouat
Copy link
Contributor Author

Skouat commented May 31, 2025

This makes me think we should also remove "Zend" as a search engine, then that would allow us to delete the zend and ezcomponents junk in includes/library

If you confirm that both can be removed, I will propose a pull request for that.

@iMattPro
Copy link
Member

iMattPro commented Jun 1, 2025

I can neither confirm nor deny. But I know on dot-com we are not using the "zend" search engine option. Not sure what maybe @DavidIQ thinks.

@DavidIQ
Copy link
Member

DavidIQ commented Jun 4, 2025

I can neither confirm nor deny. But I know on dot-com we are not using the "zend" search engine option. Not sure what maybe @DavidIQ thinks.

Should be fine 👍

@Skouat
Copy link
Contributor Author

Skouat commented Jun 4, 2025

Should be fine 👍

Good. I will check this.

And how about this PR? Is it good for merging?

@Skouat
Copy link
Contributor Author

Skouat commented Jun 13, 2025

Hi @DavidIQ
Same than the other PR…
Are there any other changes to apply? Or is it okay for merging?
Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants