Skip to content

Conversation

@n10l
Copy link
Contributor

@n10l n10l commented May 19, 2018

Right now maintenance page breaks for me due to missing react import in Maintenance.jsx and /error and /404 error pages are not visible due to screwed up routing in src/routing/settings page. My this pull request fixes both issues to make these pages working. Please give feedback.

n10l added 2 commits May 20, 2018 01:52
…is page MAINTENANCE_MODE should be set to true in src/config/constants.js near line 321
…xported instead of just routes, which is capturing any unmatched route and making any route below this unaccessible (e.g. /404 or /error). Also, comma after jsx comments is required because jsx is not contained inside any function
@RishiRajSahu RishiRajSahu requested review from gondzo and removed request for gondzo May 20, 2018 09:37
Copy link
Contributor

@RishiRajSahu RishiRajSahu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@RishiRajSahu RishiRajSahu merged commit 8b6cc79 into topcoder-archive:dev May 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants