[C-1883] Re-direct app.crowd.dev to app.crowd.dev/auth/signup (if user is not logged in)

[jonathimer]

Problem

When visiting https://app.crowd.dev/, users can re-directed to https://app.crowd.dev/auth/signin if they are not logged in

Solution

Users should be re-directed to https://app.crowd.dev/auth/signup if they are not logged in to increase conversion

_{From SyncLinear.com | C-1883}

[peoray]

@jonathimer @joanagmaia is this still a valid issue? If so, please provide more context as I cannot reproduce this

[a0m0rajab]

I think it's resolved.

[skwowet]

@peoray @a0m0rajab, I think the issue still holds up. To perform a quick test, I would do the following steps:

1. Launch an incognito browser window
2. Navigate to https://app.crowd.dev/
3. We're still being redirected to https://app.crowd.dev/auth/signin

However, the improvement suggested here aims to redirect users who aren't signed in to https://app.crowd.dev/auth/signup instead.

[a0m0rajab]

@yeganathan18 with your steps if works as expected, since we are not logged in the incognito mode

[skwowet]

Exactly, so we should be changing the redirection from /signin to /signup.

[peoray]

@a0m0rajab wanna pick it up?

[a0m0rajab]

Yeah, I was thinking to pick it up, mostly the code needed to change is in this lines:

crowd.dev/frontend/src/middleware/auth/auth-guard.js

Lines 15 to 40 in e91b729

	* - User is authenticated, and both currentTenant & currentUser exist within our store (if not, redirects to /auth/signin)
	* - Email of that user is verified (if not, redirects to /auth/email-unverified)
	* - User is onboarded (if not, redirects to /onboard)
	* - User has permissions (if not, redirects to /auth/empty-permissions)
	*
	* @param to
	* @param store
	* @param router
	* @returns {Promise<*>}
	*/
	export default async function ({ to, store, router }) {
	if (!to.meta || !to.meta.auth) {
	return;
	}
	await store.dispatch('auth/doWaitUntilInit');
	const currentUser = store.getters['auth/currentUser'];
	const permissionChecker = new PermissionChecker(
	store.getters['auth/currentTenant'],
	currentUser,
	);
	if (!permissionChecker.isAuthenticated) {
	router.push({ path: '/auth/signin' });
	return;

@peoray if you like you we can collaborate on it? (yes, it's simple but yeah)

[peoray]

It's fine, you can go ahead. Just wanted to be sure :) @a0m0rajab

[a0m0rajab]

@peoray Thanks for that! If you feel to pick it up, it's okay for me too!