Remove authState.isPending

[mraible]

Question: is it possible to do !authState?.isAuthenticated instead of !authState && !authState.isAuthenticated?

[mraible]

It looks like this file needs to be fixed too.

[denysoblohin-okta]

Thanks for submitting this PR.
Yes, isPending is obsolete and should be removed.

Code !authState.isPending && should be removed completely, no need to change it to !authState because there are lines before that make this check:

samples-js-react/okta-hosted-login/src/Navbar.jsx

Lines 38 to 40 in 6702f99

	if (!authState) {
	return null;
	}

Same for this file

[mraible]

@denysoblohin-okta How should I change this line? If I remove the conditionals, will a login button still be rendered?

{!authState.isPending && !authState.isAuthenticated && <Menu.Item onClick={login}>Login</Menu.Item>}

[shuowu]

@mraible where did you see the issues with the code snippet above? this sample repo or other apps?
As optional chaining is a part of ES2020, you may need to config the transpiler to make it work.

[mraible]

@shuowu It worked for OktaDev Schematics, which generates an app with the latest version of Create React App. oktadev/schematics@2c90198