-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix Class Management #85
base: development
Are you sure you want to change the base?
Conversation
# Conflicts: # src/App.jsx # src/utils/axios/axiosConfig.js
* feat: responsive internship management (#68) * feat: implementation coordinator api in select coordinator component on create internship * feat: integration create internship api * fix: solve null internship context at internship dashboard * fix: direct user to login if token is expired * refactor: remove unused file in program management component * feat: responsive internship card component at internship dashboard * feat: responsive internship header * feat: responsive list internship at internship dashboard * feat:responsive header dashboard component * feat: responsive detail internship * refactor: remove unnecessary file * fix: header breadcumb component on all page * fix: responsive internship card title on mobile phone * feat: add limit chars for header dashboard component * fix: limit breadcumb char on header dashboard component * feat: dynamic path on header dashboard component * feat: add collapses component on detail description internship * style: change bg color on delete btn and modal * feat: responsive header on 300-400px width * feat: responsive internship card on screen less than 460 * refactor(Opportunity): refactor and standarize the code styles * fix(Opportunity): fix text rich editor render method on update opportunity * fix: payload empty after click submit at forgot password page (#69)
# Conflicts: # package-lock.json # src/App.jsx # src/components/components.js # src/enums/enums.js # src/hooks/hooks.js # src/pages/Auth/ForgotPassword.jsx # src/pages/Auth/Login.jsx # src/pages/Class/ClassDashboard.jsx # src/pages/Internship/libs/components/DetailInternship/DetailInternship.jsx # src/pages/Internship/libs/components/ListInternship/ListInternship.jsx # src/pages/Major/MajorDashboard.jsx # src/pages/School/SchoolDashboard.jsx
…management # Conflicts: # src/components/components.js # src/hooks/hooks.js # src/pages/Class/ClassDashboard.jsx # src/pages/Class/libs/components/ClassList/ClassColumns.jsx # src/pages/Class/libs/components/ClassList/ClassTable.jsx
src/context/ClassContext.jsx
Outdated
import {createContext, useState, useCallback} from "react"; | ||
import {ClassApi} from "@/utils/axios/ApiService"; | ||
import { createContext, useMemo } from "react"; | ||
import useClass from "@/hooks/class/useClass.js"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@/hooks/hooks
@@ -0,0 +1,94 @@ | |||
import {useState, useEffect, useRef} from "@/hooks/hooks"; | |||
import {Button, Checkbox} from "antd"; | |||
import {CaretDownFilled} from "@ant-design/icons"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
import {Button, Checkbox, CaretDownFilled} from @/components/components
src/pages/Dashboard/Dashboard.jsx
Outdated
@@ -0,0 +1,45 @@ | |||
import { Header } from "@/components/Dashboard/Header.jsx"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@/components/components
src/pages/Dashboard/Dashboard.jsx
Outdated
@@ -0,0 +1,45 @@ | |||
import { Header } from "@/components/Dashboard/Header.jsx"; | |||
import { useEffect, useState } from "react"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@/hooks/hooks
src/pages/Dashboard/Dashboard.jsx
Outdated
afterClose={() => setShowAlert(false)} | ||
style={{ | ||
position: 'fixed', | ||
width: '80%', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use tailwind className
src/pages/Class/libs/components/ClassList/components/ClassFilterButton.jsx
Outdated
Show resolved
Hide resolved
Don't forget to solve your conflict |
…ev-sp8-class-management # Conflicts: # src/App.jsx # src/components/components.js # src/context/ClassContext.jsx # src/pages/Class/ClassDashboard.jsx # src/pages/Class/libs/components/ClassList/ClassTable.jsx # src/pages/Class/libs/components/modals/ClassDetail.jsx # src/pages/Dashboard/Dashboard.jsx # src/utils/axios/api/Class.js
src/hooks/class/useClass.js
Outdated
@@ -0,0 +1,84 @@ | |||
import { useCallback, useState } from "react"; | |||
import { ClassApi } from "@/utils/axios/ApiService.js"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
import { ClassApi } from "@/utils/utils";
src/pages/Class/ClassDashboard.jsx
Outdated
<Header | ||
pageName="Class Dashboard" | ||
subPages={[ | ||
{ title: "Class Management", path: "/class-management" } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
import { AppRoute } from "@/enums/enums";
path: AppRoute.CLASS_MANAGEMENT
import {ClassApi, MajorApi} from "@/utils/axios/ApiService.js"; | ||
import {fetchMajors} from "@/pages/Class/libs/components/utils.js"; | ||
import {ClassApi} from "@/utils/utils"; | ||
import {fetchMajors} from "@/utils/utils"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
import {fetchMajors, ClassApi} from "@/utils/utils";
import {fetchClassData, fetchMajors} from "@/pages/Class/libs/components/utils.js"; | ||
import {ClassApi} from "@/utils/axios/ApiService.js"; | ||
import {fetchClassData, fetchMajors} from "@/utils/utils"; | ||
import {ClassApi} from "@/utils/utils"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
import {fetchClassData, fetchMajors, ClassApi} from "@/utils/utils";
…management # Conflicts: # src/components/components.js # src/context/contexts.js
src/hooks/class/useData.js
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this file is not imported in any file
@@ -0,0 +1,79 @@ | |||
import {useState, useEffect, useRef} from "@/hooks/hooks.js"; | |||
import {Button, Checkbox, CaretDownFilled} from "@/components/components.js"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove ".js" from import statement
No description provided.