search

EasyEngine / site-type-wp

Adds 'wp' site type to EasyEngine
12 stars 14 forks source link

Remove password check #178

Closed kirtangajjar closed 4 years ago

kirtangajjar commented 4 years ago

Remove password check as it's interfering with the site clone.

mrrobot47 commented 4 years ago

How is it interfering?

kirtangajjar commented 4 years ago

I thought this PR was needed during the testing of site clone. I found existing sites that had passwords that violated EE's password rules. I thought that the strict password's logic was written long ago and somehow due to a bug it was being bypassed, so I thought it would be best to remove this logic. After having a discussion with @mrrobot47, I came to know that these features were added recently, and the sites were created before this feature was added. So I am closing this PR as it does not look necessary now.