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.

Sign up for GitHub

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

Jump to bottom

[ACEAO-5766] Changes for inactive site redirection #165

Open

cb-manojpathak wants to merge 2 commits into cms-dep from bugfix/ACEAO-5766

cb-manojpathak commented May 29, 2020 •

edited

Loading

Please follow the below steps:

Verify the latest changes(two files) and apply the same change in CMSPencileblue repo. Changes will be require manually on CMSPencileblue repo for below files:

node_modules/@cbtn/pencilblue/include/http/middleware/routing.js
node_modules/@cbtn/pencilblue/include/http/middleware/index.js

Run the app.
Verify the changes.

Testing Steps:

Open https://premium.lvh.me:8080/admin/sites
Deactivate Premium or Bravo.
Open below URLs to verify expected behaviour.

https://premium.lvh.me:8080/bower_components/fontawesome/css/font-awesome.min.css
https://premium.lvh.me:8080/bower_components/fontawesome/css/
https://premium.lvh.me:8080/bower_components/fontawesome/
https://premium.lvh.me:8080/bower_components/

All above URLs should redirect on https://www.careerbuilder.com/browse?TNNotFound=true
PFB the screenshot:

cb-manojpathak added 2 commits

May 28, 2020 19:53


          [ACEAO-5766]: Changes for redirection

da44a4c


          changes with index.js

5a29f6b

cb-manojpathak requested review from vokidya and lyliuCB

May 29, 2020 07:03

cb-manojpathak changed the title ~~Bugfix/aceao 5766~~ [ACEAO-5766] Changes for inactive site redirection

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet