search

department-of-veterans-affairs / veterans-employment-center

Veterans Employment Center
https://www.vets.gov/veterans-employment-center
5 stars 4 forks source link

Pv remove saml #380

Closed patrickvinograd closed 7 years ago

patrickvinograd commented 7 years ago

Rebased after upgrading rails. Removes all saml/DS-Logon-related code and dependencies.

Plan is to proactively merge this to master in preparation for doing some regression testing, hopefully tomorrow.

ayaleloehr commented 7 years ago

All the SAML things LGTM.

You pulled out app/views/static_pages/_jobsearch.html, which I think is related to the next PR you're likely to pulling out all the job search functionality? That's the only file that felt a little off relative to this PR; not a problem though because it will be removed either way!