Skip to content

Commit 446c304

Browse files
committed
Merge branch 'development' into chore/refactor
2 parents b67b657 + a24d33d commit 446c304

File tree

8 files changed

+99
-106
lines changed

8 files changed

+99
-106
lines changed
Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
node_modules
2+
tests
3+
.env*
4+
*.md
Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
FROM node:20-alpine
2+
3+
WORKDIR /question-service
4+
5+
COPY package*.json ./
6+
7+
RUN npm ci
8+
9+
COPY . .
10+
11+
EXPOSE 3000
12+
13+
CMD ["npm", "start"]

backend/question-service/package-lock.json

Lines changed: 19 additions & 87 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

backend/user-service/.dockerignore

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
node_modules
2+
tests
3+
.env*
4+
*.md

backend/user-service/Dockerfile

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
FROM node:20-alpine
2+
3+
WORKDIR /user-service
4+
5+
COPY package*.json ./
6+
7+
RUN npm ci
8+
9+
COPY . .
10+
11+
EXPOSE 3001
12+
13+
CMD ["npm", "start"]

docker-compose.yml

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,25 @@
1+
services:
2+
user-service:
3+
image: peerprep/user-service
4+
build: ./backend/user-service
5+
env_file: ./backend/user-service/.env
6+
ports:
7+
- 3001:3001
8+
networks:
9+
- peerprep-network
10+
restart: on-failure
11+
question-service:
12+
image: peerprep/question-service
13+
build: ./backend/question-service
14+
env_file: ./backend/question-service/.env
15+
ports:
16+
- 3000:3000
17+
depends_on:
18+
- user-service
19+
networks:
20+
- peerprep-network
21+
restart: on-failure
22+
23+
networks:
24+
peerprep-network:
25+
driver: bridge

frontend/src/pages/QuestionList/index.tsx

Lines changed: 18 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -66,6 +66,17 @@ const QuestionList: React.FC = () => {
6666
);
6767
};
6868

69+
const updateQuestionList = () => {
70+
getQuestionList(
71+
page + 1, // convert from 0-based indexing
72+
rowsPerPage,
73+
searchFilter,
74+
complexityFilter,
75+
categoryFilter,
76+
dispatch
77+
);
78+
};
79+
6980
// For handling edit / delete menu
7081
const [targetQuestion, setTargetQuestion] = useState<string | null>(null);
7182
const [menuAnchor, setMenuAnchor] = useState<HTMLElement | null>(null);
@@ -106,25 +117,11 @@ const QuestionList: React.FC = () => {
106117

107118
toast.success(SUCCESS_QUESTION_DELETE);
108119
getQuestionCategories(dispatch);
109-
getQuestionList(
110-
page + 1, // convert from 0-based indexing
111-
rowsPerPage,
112-
searchFilter,
113-
complexityFilter,
114-
categoryFilter,
115-
dispatch
116-
);
117-
};
118-
119-
const updateQuestionList = () => {
120-
getQuestionList(
121-
page + 1, // convert from 0-based indexing
122-
rowsPerPage,
123-
searchFilter,
124-
complexityFilter,
125-
categoryFilter,
126-
dispatch
127-
);
120+
if (state.questionCount % rowsPerPage !== 1 || page === 0) {
121+
updateQuestionList();
122+
} else {
123+
setPage(page - 1);
124+
}
128125
};
129126

130127
useEffect(() => {
@@ -137,8 +134,10 @@ const QuestionList: React.FC = () => {
137134
} else {
138135
updateQuestionList();
139136
}
137+
// eslint-disable-next-line react-hooks/exhaustive-deps
140138
}, [searchFilter, complexityFilter, categoryFilter]);
141139

140+
// eslint-disable-next-line react-hooks/exhaustive-deps
142141
useEffect(() => updateQuestionList(), [page]);
143142

144143
// Check if the user is admin

frontend/vite.config.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,4 +6,7 @@ import svgr from "vite-plugin-svgr";
66
export default defineConfig({
77
plugins: [react(), svgr()],
88
optimizeDeps: { include: ["@mui/material", "@mui/icons-material"] },
9+
server: {
10+
host: true,
11+
},
912
});

0 commit comments

Comments
 (0)