remove NEW_HOME_PAGE flag

RESOLVED FIXED

Status

Webmaker
webmaker.org
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: pomax, Assigned: inactivate account for kate)

Tracking

Details

Attachments

(1 attachment)

(Reporter)

Description

4 years ago
Now that we've made the transition to the new homepage in production, we should remove the NEW_HOME_PAGE environment variable from env.dist and compact the code path for rendering the homepage, since it no longer needs to rely on it.
(Assignee)

Comment 1

4 years ago
Created attachment 8341107 [details] [review]
https://github.com/mozilla/webmaker.org/pull/532
Attachment #8341107 - Flags: review?(dale)

Comment 2

4 years ago
Comment on attachment 8341107 [details] [review]
https://github.com/mozilla/webmaker.org/pull/532

Code looks good, and works.

Travis is failing though. Fix them Travis errors, rebase, and then you're good to push!
Attachment #8341107 - Flags: review?(dale) → review+

Comment 3

4 years ago
Commit pushed to master at https://github.com/mozilla/webmaker.org

https://github.com/mozilla/webmaker.org/commit/9aa6dfc133f4dd4ad47cab0bf988ebf423b5743d
Fix bug 943576 - Remove new home page flag

Updated

4 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.