8000 [5.0] Remove outdated captcha for new installation by tecpromotion · Pull Request #2940 · joomlagerman/joomla · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

[5.0] Remove outdated captcha for new installation #2940

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

Merged
merged 5 commits into from
Sep 4, 2023

Conversation

tecpromotion
Copy link
Member

Pull Request für Issue #2931 .

Zusammenfassung der Änderungen

remove 4 .ini files

Wo wird der Sprachstring angezeigt / Wie kann getestet werden

backend

@tecpromotion tecpromotion requested a review from heelc29 September 3, 2023 13:25
@tecpromotion tecpromotion self-assigned this Sep 3, 2023
@tecpromotion tecpromotion requested review from a team and zero-24 and removed request for a team September 3, 2023 13:25
@tecpromotion tecpromotion added this to the 5.0.0v1 milestone Sep 3, 2023
@tecpromotion tecpromotion linked an issue Sep 3, 2023 that may be closed by this pull request
script.php Outdated
Comment on lines 409 to 410
'/administrator/language/de-DE/plg_captcha_recaptcha_invisible.ini',
'/administrator/language/de-DE/plg_captcha_recaptcha_invisible.sys.ini',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Das recaptcha_invisible wird ja nicht deinstalliert, sondern nur entsperrt und bleibt weiterhin installiert.
Problem: Da wir es ja nicht mitbekommen, ob im nachhinein das Plugin (manuell) deinstalliert wird, würden bei einer Deinstallation die deutschen Übersetzungen weiterhin verbleiben.

Passt es dann, dass das Plugin in J5 nur noch in englischer Sprache vorhanden ist - es wird ja auch nicht mehr vom Core mit "Updates" versorgt!?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ja das betrifft nur die Neuinstallation

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In Crowdin wird das dann auch automatisch gelöscht sein.

Was meinst du @zero-24 ?
Wir wollen wir das lösen?

Copy link
Member
@zero-24 zero-24 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ja crowdin wird die Datei löschen. Wir haben zwei Möglichkeiten. Entweder behalten wir die Sprachdateien oder wir nehmen die invisible nur noch nicht ins lösch script mit auf und löschen die dann erst mit 6.x

@tecpromotion
Copy link
Member Author

Ja crowdin wird die Datei löschen. Wir haben zwei Möglichkeiten. Entweder behalten wir die Sprachdateien oder wir nehmen die invisible nur noch nicht ins lösch script mit auf und löschen die dann erst mit 6.x

Das ist doch eine gute Idee. Heißt bei Neuinstallationen wird sie nicht mehr mitinstalliert und mit 6 aktiv gelöscht!

Co-authored-by: Tobias Zulauf <zero-24@users.noreply.github.com>
@tecpromotion tecpromotion requested a review from zero-24 September 4, 2023 11:18
@tecpromotion tecpromotion merged commit 27dcaa8 into joomlagerman:5.0-dev Sep 4, 2023
@tecpromotion tecpromotion deleted the issue-2931 branch September 4, 2023 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

[5.0] Remove outdated captcha for new installation
3 participants
0