Skip to content

Commit cf36fca

Browse files
committed
remove client-side redirects
1 parent 5a51c70 commit cf36fca

File tree

3 files changed

+1
-70
lines changed

3 files changed

+1
-70
lines changed

apps/frontend/src/app/profile/page.tsx

Lines changed: 0 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@ import {
2020
ValidateUser,
2121
VerifyTokenResponseType,
2222
} from "../services/user";
23-
import { useRouter } from 'next/navigation';
2423

2524
interface ProfilePageProps {}
2625

@@ -33,30 +32,6 @@ const ProfilePage = (props: ProfilePageProps): JSX.Element => {
3332
const [refresh, setRefresh] = useState<boolean>(false);
3433
const [messageApi, contextHolder] = message.useMessage();
3534

36-
// used to check if user JWT is verified
37-
38-
const [userId, setUserId] = useState<string | undefined>(undefined);
39-
const [isAdmin, setIsAdmin] = useState<boolean | undefined>(undefined);
40-
41-
const router = useRouter();
42-
43-
useLayoutEffect(() => {
44-
var isAuth = false;
45-
46-
ValidateUser().then((data: VerifyTokenResponseType) => {
47-
setUserId(data.data.id);
48-
setEmail(data.data.email);
49-
setUsername(data.data.username);
50-
setIsAdmin(data.data.isAdmin);
51-
isAuth = true;
52-
}).finally(() => {
53-
if(!isAuth){
54-
// cannot verify
55-
router.push('/login'); // Client-side redirect using router.push
56-
}
57-
});
58-
}, [router])
59-
6035
useEffect(() => {
6136
ValidateUser().then((data: VerifyTokenResponseType) => {
6237
form.setFieldsValue({

apps/frontend/src/app/question/[id]/page.tsx

Lines changed: 0 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -57,33 +57,7 @@ export default function Home() {
5757
const [categories, setCategories] = useState<string[]>([]); // Store the selected filter categories
5858
const [description, setDescription] = useState<string | undefined>(undefined);
5959
const [selectedItem, setSelectedItem] = useState("python"); // State to hold the selected language item
60-
61-
// used to check if user JWT is verified
62-
63-
const [userId, setUserId] = useState<string | undefined>(undefined);
64-
const [email, setEmail] = useState<string | undefined>(undefined);
65-
const [username, setUsername] = useState<string | undefined>(undefined);
66-
const [isAdmin, setIsAdmin] = useState<boolean | undefined>(undefined);
67-
68-
const router = useRouter();
6960

70-
useLayoutEffect(() => {
71-
var isAuth = false;
72-
73-
ValidateUser().then((data: VerifyTokenResponseType) => {
74-
setUserId(data.data.id);
75-
setEmail(data.data.email);
76-
setUsername(data.data.username);
77-
setIsAdmin(data.data.isAdmin);
78-
isAuth = true;
79-
}).finally(() => {
80-
if(!isAuth){
81-
// cannot verify
82-
router.push('/login'); // Client-side redirect using router.push
83-
}
84-
});
85-
}, [router])
86-
8761
// When code editor page is initialised, fetch the particular question, and display in code editor
8862
useEffect(() => {
8963
if (!isLoading) {

apps/frontend/src/app/question/page.tsx

Lines changed: 1 addition & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,6 @@ import {
4040
import Link from "next/link";
4141
import TextArea from "antd/es/input/TextArea";
4242
import { ValidateUser, VerifyTokenResponseType } from "../services/user";
43-
import { useRouter } from "next/navigation";
4443

4544
/**
4645
* defines the State of the page whe a user is deleing an object. Has 3 general states:
@@ -118,31 +117,14 @@ export default function Home() {
118117

119118
// used to check if user JWT is verified
120119

121-
const [userId, setUserId] = useState<string | undefined>(undefined);
122-
const [email, setEmail] = useState<string | undefined>(undefined);
123-
const [username, setUsername] = useState<string | undefined>(undefined);
124120
const [isAdmin, setIsAdmin] = useState<boolean | undefined>(undefined);
125121

126-
const router = useRouter();
127-
128122
useLayoutEffect(() => {
129-
var isAuth = false;
130-
131123
ValidateUser()
132124
.then((data: VerifyTokenResponseType) => {
133-
setUserId(data.data.id);
134-
setEmail(data.data.email);
135-
setUsername(data.data.username);
136125
setIsAdmin(data.data.isAdmin);
137-
isAuth = true;
138126
})
139-
.finally(() => {
140-
if (!isAuth) {
141-
// cannot verify
142-
router.push("/login"); // Client-side redirect using router.push
143-
}
144-
});
145-
}, [router]);
127+
}, []);
146128

147129
const handleEditClick = (index: number, question: Question) => {
148130
// Open the modal for the specific question

0 commit comments

Comments
 (0)