Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Pre-Upgrade Compatibility Check on Plugins That Could Kill Joomla Upgrade #31200
Pre-Upgrade Compatibility Check on Plugins That Could Kill Joomla Upgrade #31200
Changes from 50 commits
fa683c4
ed9d1fa
d1c4498
f7d84a9
acc67ec
1913597
6ec5735
bcce3c1
8dbde56
78d9cdf
b978657
3ad4e33
6615f87
49940f7
8bc2f9b
7e3260f
0105612
7640546
c282bad
985c972
5390aba
a1f56d0
0081f5a
871d0b6
8836f0c
4ce5907
146c7fd
2a59820
9c16dad
b3e691a
4e52937
63f12cc
44e02fc
f90f81f
32eaa1f
1a119b7
b440563
99db19e
60eaca4
4d12a0b
d6a542b
4c00b6b
fcd2f83
940092f
a1f197e
e8b1112
009778f
fb7d407
f96229e
5e833c4
84ae724
b7e1b24
1de7224
dbb9c22
4e5f2bf
e3b9278
c67e078
4358d4f
5ec4417
666a960
bf8d43d
0f9108e
1f2d63b
bc0816d
9005b02
0802d72
a9f3dea
6c00eec
7de127e
2786566
9639b37
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not really sure when this is displayed as I couldnt see it BUT need to be change the language or it might be interpreted as saying that joomla is checking the code in the plugins
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you switch to the 'live update' tab as soon as you load the update page then its shown next to the list of plugins.
It only really shows up if the update checks are being slow and is mean to encourage patience! 'The system is currently checking' was meant to convey that the website is doing something right now and the user needs to wait.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we not display this on the same page?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I placed the information directly above the "submit" button to make it really obvious to the user. The downside, as you point out, is the information in 2 different places.