Octavi00 commented on code in PR #11812:
URL: https://github.com/apache/gravitino/pull/11812#discussion_r3576261464
##########
web-v2/web/src/app/page.js:
##########
@@ -19,6 +19,16 @@
'use client'
+import { useEffect } from 'react'
+
+import { useRouter } from 'next/navigation'
+
export default function Home() {
- return <></>
+ const router = useRouter()
+
+ useEffect(() => {
+ router.replace('/metalakes')
Review Comment:
This was more of an issue with the URL adjustment that I had asked about
previously. Basically, this is unrelated to my login function, and more so UX.
If a user loads the main page, http://localhost:8090, they are greeted with a
blank white page. Since I was previously explained that the main page for
gravitino is the metalakes page, I had decided that it would be nice to include
a redirect to the metalake page, which is the functional main page. This would
happen regardless of authentication being enabled.
The nice thing about this change is that when auth is enabled, if the user
is sent to metalakes page, they are then sent to login due to the metalakes
page being locked without a token.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
