silverstripe / silverstripe-mfa

MultiFactor Authentication for Silverstripe CMS
BSD 3-Clause "New" or "Revised" License
11 stars 24 forks source link

DEP Remove silverstripe/security-extensions dependency #482

Closed GuySartorelli closed 1 year ago

GuySartorelli commented 1 year ago

That functionality has been moved into the core modules (once the other PRs in the parent issue have been merged).

NOTE: The js test will fail until both the admin and webpack config PRs are merged. Once the webpack config PR is merged, this PR needs to be updated with yarn upgrade @silverstripe/webpack-config and the updated yarn.lock committed.

Parent issue

GuySartorelli commented 1 year ago

The webpack config PR was merged, tagged, and published. I've now updated this PR to correctly include it. The only change to the dist was that more vendor packages are excluded.