Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Podpora pro PHP 7.4 a 8.0 #40

Merged
merged 1 commit into from
Mar 18, 2021
Merged

Podpora pro PHP 7.4 a 8.0 #40

merged 1 commit into from
Mar 18, 2021

Conversation

MilanPala
Copy link
Contributor

@MilanPala MilanPala commented Mar 5, 2021 •
edited
Loading

Původně jsem chtěl jen přidat PHP 8, ale phpcs mi opravil typy propert, tak jsem rovnou dropnul starší PHP. Nějaké testy na to jsou, ale jinak nevím, jestli to je 100% úprava. Ty anotační typy propert nebyly vždy správně zapsané s ohledem na to, jak fungovaly.

recaptchu jsem potřeboval aktualizovat na 3.4, tak jsem ji rovnou povolil celou ve větvi 3.*.

@MilanPala MilanPala force-pushed the php8 branch 6 times, most recently from df3460a to 65572d4 Compare March 5, 2021 16:58
@MilanPala MilanPala marked this pull request as ready for review March 5, 2021 16:59
@MilanPala MilanPala requested review from Jakub-Fajkus, Spamercz and michalhlavka and removed request for Jakub-Fajkus March 5, 2021 17:01
@MilanPala MilanPala changed the title Podpora pro PHP 8.0 Podpora pro PHP 7.4 a 8.0 Mar 5, 2021
@Jakub-Fajkus
Copy link
Contributor

Jakub-Fajkus commented Mar 9, 2021 •
edited
Loading

  • Mas nejak sepsane/otestovane, co dela verze recaptchy 3.4.x oproti 3.3.x? - ja tam nevidim zadne zmeny, ktere by staly za rec
  • Upravuje se tam nejaky JS, ovlivni nas to nejak?
  • Nova zavislost na contributte/di - muze nas nejak ovlivnit, omezovat apod.? - vypada to ze nam to vadit nebude, nezavadi to zadne dalsi zbytecne zavislosti.

EDIT: upravy v kodu zde v PR jsou blbosti, to je za mne v pohode

@Jakub-Fajkus
Copy link
Contributor

@zipper V nove verzi recaptchy upravovali trochu JS soubory, viz contributte/reCAPTCHA@v3.3.1...v3.4.0#diff-1157237af5abbbf6557fb8e7ce7be367032ac0399618732c1081e9e1cf1643d3 ... pouzivame ty soubory vubec? Muzou nam ty zmeny v jsku nejak vadit?

@zipper
Copy link
Contributor

zipper commented Mar 9, 2021 •
edited
Loading

@Jakub-Fajkus Jejich JS nevyužíváme, mámě napsaný vlastní, především i kvůli podpoře pro lazy load recaptcha JS. Změny jsem si prošel a myslím, že se nás nijak netýkají.

Řeší tam změnu doby, kdy se volá execute, což měli chybně už při load stránky. U nás to voláme správně až při odeslání formuláře.

@MilanPala
Copy link
Contributor Author

Mas nejak sepsane/otestovane, co dela verze recaptchy 3.4.x oproti 3.3.x?

Taky nevidím nic, co mění funkcionalitu.

Nova zavislost na contributte/di

Podle commitu si to přidali z nějakého jejich důvodu bez ohledu na funkčnost, ale s tím nic asi neuděláme.

@MilanPala MilanPala force-pushed the php8 branch 2 times, most recently from a221014 to 7eb86f6 Compare March 9, 2021 22:32
@MilanPala MilanPala merged commit c396b39 into master Mar 18, 2021
@MilanPala MilanPala deleted the php8 branch March 18, 2021 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Jakub-Fajkus Jakub-Fajkus Jakub-Fajkus approved these changes

@Spamercz Spamercz Awaiting requested review from Spamercz

@michalhlavka michalhlavka Awaiting requested review from michalhlavka

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@MilanPala @Jakub-Fajkus @zipper