Prefer signup over login page when requesting authentication

This commit is contained in:
Kimmo Puputti 2017-05-03 15:45:08 +03:00
parent 5676103177
commit c3c0a87468
3 changed files with 24 additions and 23 deletions

View file

@ -65,7 +65,7 @@ class RouteComponentRenderer extends Component {
location={location}
flattenedRoutes={flattenedRoutes}
/>
: <NamedRedirect name="LogInPage" state={{ from: match.url }} />;
: <NamedRedirect name="SignupPage" state={{ from: match.url }} />;
}
}

View file

@ -41,8 +41,8 @@ describe('Application', () => {
'/s/map': 'Search page: map',
'/l/listing-title-slug/1234': 'Loading listing data',
'/u/1234': 'Profile page with display name: 1234',
'/login': 'Log in',
'/signup': 'Signup',
'/login': 'Login',
'/signup': 'Sign up',
'/password': 'Request new password',
'/password/forgotten': 'Request new password',
'/password/change': 'Type new password',
@ -60,25 +60,26 @@ describe('Application', () => {
});
it('server renders redirects for pages that require authentication', () => {
const defaultAuthPath = '/signup';
const urlRedirects = {
'/l/new': '/login',
'/l/listing-title-slug/1234/edit': '/login',
'/l/listing-title-slug/1234/checkout': '/login',
'/u/1234/edit': '/login',
'/inbox': '/login',
'/inbox/orders': '/login',
'/inbox/sales': '/login',
'/order/1234': '/login',
'/order/1234/discussion': '/login',
'/order/1234/details': '/login',
'/sale/1234': '/login',
'/sale/1234/discussion': '/login',
'/sale/1234/details': '/login',
'/listings': '/login',
'/account': '/login',
'/account/contact-details': '/login',
'/account/payout-preferences': '/login',
'/account/security': '/login',
'/l/new': defaultAuthPath,
'/l/listing-title-slug/1234/edit': defaultAuthPath,
'/l/listing-title-slug/1234/checkout': defaultAuthPath,
'/u/1234/edit': defaultAuthPath,
'/inbox': defaultAuthPath,
'/inbox/orders': defaultAuthPath,
'/inbox/sales': defaultAuthPath,
'/order/1234': defaultAuthPath,
'/order/1234/discussion': defaultAuthPath,
'/order/1234/details': defaultAuthPath,
'/sale/1234': defaultAuthPath,
'/sale/1234/discussion': defaultAuthPath,
'/sale/1234/details': defaultAuthPath,
'/listings': defaultAuthPath,
'/account': defaultAuthPath,
'/account/contact-details': defaultAuthPath,
'/account/payout-preferences': defaultAuthPath,
'/account/security': defaultAuthPath,
};
forEach(urlRedirects, (redirectPath, url) => {
const context = {};

View file

@ -126,13 +126,13 @@ const routesConfiguration = [
{
path: '/login',
exact: true,
name: 'LogInPage',
name: 'LoginPage',
component: props => <AuthenticationPage {...props} tab="login" />,
},
{
path: '/signup',
exact: true,
name: 'SignUpPage',
name: 'SignupPage',
component: props => <AuthenticationPage {...props} tab="signup" />,
},
{