Use getSession's status within auth and account layouts to render/re-direct

[madhurjain]

If we visit the settings page https://demo.nextacular.co/account/settings and hit refresh/reload, we end up on the https://demo.nextacular.co/account page. This happens because on the first render of the AccountLayout, the getSession()'s data object is not ready and the user gets incorrectly re-directed to /auth/login. Within the AuthLayout, the getSession()->data is checked again, it is found to be valid and then the user is re-directed to https://demo.nextacular.co/account/.

This pull-request attempts to fix the issue by waiting until getSession returns a valid data and this is checked using the status object returned by getSession() instead of the data object. This also fixes #28 and #20

Please note: Since there are no existing test cases, I couldn't test these changes on the nextacular demo site but I did test it out on the code base for my personal project. I would be happy to revise the changes if required.

[vercel]

@madhurjain is attempting to deploy a commit to the Nextacular Team on Vercel.

A member of the Team first needs to authorize it.

[arjayosma]

Hi @madhurjain, I'll review this PR now. Thanks for the amazing job.

[arjayosma]

Code works and is a lifesaver.

If you wish to use the data object, ensure that you access it after status === 'authenticated'.

[WebDev-Akhil]

@madhurjain @arjayosma

I don't think this PR fixed #28.

The workspace dashboard stills returns white page (eg. http://localhost:3000/account/test1)

I think this is because of useWorkspace() not loading when the user reloads the page.

In src/pages/account/[workspaceSlug]/index.js, if you replace the current return with something static (example code below) then it will work but if you want to consume useWorkspace() it doesn't work

the below works on reload (as it is static)

return (
  <AccountLayout>
    <Meta title={`Nextacular - some hardcoded name | Dashboard`} />
    <Content.Title
      title="some hardcoded name"
      subtitle="This is your project's workspace"
    />
    <Content.Divider />
    <Content.Container />
  </AccountLayout>
);

the below doesn't works (as it consumes useWorkspace() and, on page reload it is not being fetched)

return (
    workspace && (
      <AccountLayout>
        <Meta title={`Nextacular - ${workspace.name} | Dashboard`} />
        <Content.Title
          title={workspace.name}
          subtitle="This is your project's workspace"
        />
        <Content.Divider />
        <Content.Container />
      </AccountLayout>
    )
  );

Correct me if I'm wrong