-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Remove deprecated ajaxDie method and deprecated hooks #35283
Remove deprecated ajaxDie method and deprecated hooks #35283
Conversation
Before we can continue to have green tests, we must first merge, release and updateincomposer the three related modules PR. It fails on newsletter and themecusto. :-) |
71f857d
to
0d0329d
Compare
@Hlavtox , could you check the conflicts and fix them please ? 😄 |
c0e8cda
0d0329d
to
c0e8cda
Compare
Rebased, can you reapprove please @FabienPapet @tleon @kpodemski 🙏 |
@Hlavtox UI tests are green, could you please relaunch them? |
c0e8cda
to
36b98c3
Compare
Rebased, new tests - https://github.com/Hlavtox/ga.tests.ui.pr/actions/runs/8984005226 |
@nicosomb @kpodemski Tests finally green after rebase, can we merge? :-) |
Auto tests are ok, no need to test removal of the deprecated code. |
Related PRs
Removed methods and changes
classes/controller/Controller.php - removed ajaxDie method, removed calls to actionAjaxDieBefore and actionBeforeAjaxDie
classes/Hook.php - deleted removed hook actionAjaxDieBefore
Database changes required for upgraded installs