User tests: Successful: Unsuccessful:
Pull Request is a redo for #8513
With some additions:
Status | New | ⇒ | Pending |
Labels |
Added:
?
?
|
Category | ⇒ | JavaScript |
Easy | No | ⇒ | Yes |
@zero-24 these are the default rules
defaultOptions.rules.activated = {
wordNotEmail: true,
wordLength: true,
wordSimilarToUsername: true,
wordSequences: true,
wordTwoCharacterClasses: false,
wordRepetitions: false,
wordLowercase: true,
wordUppercase: true,
wordOneNumber: true,
wordThreeNumbers: true,
wordOneSpecialChar: true,
wordTwoSpecialChar: true,
wordUpperLowerCombo: true,
wordLetterNumberCombo: true,
wordLetterNumberCharCombo: true
};
So there are a lot of options/combinations but the rules are specific to 1,2 or 3 uppercase,number,special character so overriding these we can justify the rule for 1,2 or 3 e.g. special characters.
Thanks for the test instructions
What are character classes?
Use different character classes
Will not install. Installer not loading correctly no header etc in the html
I have tested this item unsuccessfully on 7f90087
Category | JavaScript | ⇒ | Installation JavaScript Language & Strings |
Labels |
This PR has received new commits.
CC: @brianteeman
This PR has received new commits.
CC: @brianteeman
This PR has received new commits.
CC: @brianteeman
I have tested this item unsuccessfully on 4fad4f2
Go to installation - complete step 1 and click next to go to step 2
Decide you want to change the password so click on the first tab to return to step 1
Password script is no longer in use on this page
Also if your confirm password is not match the password the javascript is no longer loaded
If when registering a user you get a message such as
Your password cannot contain your username
Then I would expect not to be able to save the user with that password
Currently you can which kind of defeats the object of this to me
This is not ready for testing at all. Not even sure if it usable in its
current form
On 10 May 2016 at 13:54, Jelle Kok notifications@github.com wrote:
@zero-24 https://github.com/zero-24 srry I thought you forked his pr.
@test https://github.com/test I can still use admin - admin
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#10377 (comment)
Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/
@brianteeman was the previous behaviour like the one you are describing above or was it just an indication? If not I guess you are right and we should match the old behaviour
This PR has received new commits.
CC: @brianteeman
This PR has received new commits.
CC: @brianteeman
I have tested this item unsuccessfully on dfdee29
Problems I reported above about the installation are still not fixed
Sorry I wont be wasting my time testing this again.
Status | Pending | ⇒ | Information Required |
Labels |
Hello @dgt41
Thank you for your contribution.
The last comment here was on May 10th. Can you update this PR please?
If no reply is received within 4 weeks we will close this issue.
Thanks for understanding!
I will redo this with own script to match the exact old behaviour. Thanks
Status | Information Required | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-06-24 23:07:21 |
Closed_By | ⇒ | dgt41 |
Important for testers: Please test the installation process of Joomla! because we've encountered an issue during the Code Sprint in Manchester after we've merged the previous state of this PR (maybe it is already fixed?). Thanks!