merge main to dev-deployment #101
Conversation
…xing a typo in Sidebar.jsx
* working * added search and pagination to donation tracking table * CSS done for DTT * updated styles * Fixed checkbox effect on all content rows * Fixes #44 pagination, search, and css added * fixed bug where page num did not update when search happened --------- Co-authored-by: Sean Kelman <[email protected]>
* implement_CSS_for_supply_request * revision for implement CSS for supply request * revision for implement CSS for supply request
Added pagination and add/edit/delete item
…s-Donation-History-Table Pagination, Search, CSS for Business Donation History Table Resolved
add firebase auth
Approve deny button
…/github.com/ctc-uci/fph-frontend into update_view_business_and_edit_business_form
… coochie! coochie coochie coochie coochie coochie coochie
Fix admin dash - merge
| } | ||
| }; | ||
| checkIsAdmin(); | ||
| }, []); |
There was a problem hiding this comment.
React Hook useEffect has missing dependencies: 'isAdmin' and 'navigate'. Either include them or remove the dependency array.
| } catch (error) { | ||
| console.error('Error fetching data:', error); | ||
| } | ||
| }, [currentTab, currentPageNum, searchTerm]); |
There was a problem hiding this comment.
React Hook useEffect has a missing dependency: 'loadInfo'. Either include it or remove the dependency array.
| } | ||
| }; | ||
| getData(); | ||
| }, [isOpen]); |
There was a problem hiding this comment.
React Hook useEffect has a missing dependency: 'data'. Either include it or remove the dependency array. If 'setNameData' needs the current value of 'data', you can also switch to useReducer instead of useState and read 'data' in the reducer.
| setNotification(response.data); | ||
| } | ||
| fetchNotifications(); | ||
| }, [isAdmin, navigate]); |
There was a problem hiding this comment.
React Hook useEffect has a missing dependency: 'backend'. Either include it or remove the dependency array.
| duration: 3000, | ||
| isClosable: true, | ||
| }); | ||
| }, [error]); |
There was a problem hiding this comment.
React Hook useEffect has a missing dependency: 'toast'. Either include it or remove the dependency array.
| useEffect(() => { | ||
| getBusinessName(); | ||
| }, | ||
| [backend]); |
There was a problem hiding this comment.
React Hook useEffect has a missing dependency: 'getBusinessName'. Either include it or remove the dependency array.
| } | ||
|
|
||
| getData(); | ||
| }, []); |
There was a problem hiding this comment.
React Hook useEffect has missing dependencies: 'backend', 'id', and 'notification.length'. Either include them or remove the dependency array.
| }; | ||
| useEffect(() => { | ||
| fetchRequest(); | ||
| }, [id]); |
There was a problem hiding this comment.
React Hook useEffect has a missing dependency: 'fetchRequest'. Either include it or remove the dependency array.
|
|
||
| const AuthContext = React.createContext(); | ||
|
|
||
| export function useAuth() { |
There was a problem hiding this comment.
Fast refresh only works when a file only exports components. Use a new file to share constants or functions between components.
| useEffect(() => { | ||
| setIsAuthenticated(currentUser !== null); | ||
| setIsLoading(false); | ||
| }, []); |
There was a problem hiding this comment.
React Hook useEffect has a missing dependency: 'currentUser'. Either include it or remove the dependency array.
No description provided.