-
Notifications
You must be signed in to change notification settings - Fork 2
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
Execom and alumini data #104
base: master
Are you sure you want to change the base?
Conversation
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.
The changes look good, but there are lot of unused variables and SocietyExecom
Component is unnessarily worked on which was supposed to be done by @sannidhi-s-shetty, update the merging branch to capExecom
@@ -0,0 +1,63 @@ | |||
import React, { useState, useEffect } 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.
This file acts as a component, better to move to Components
folder
|
||
const [member, setMember] = useState([]); | ||
|
||
useEffect(() => { |
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.
This functionality is being developed by @sannidhi-s-shetty, please coordinte with her.
<AlumniAccordions members={alumni.compsoc} color="rgb(80 161 99)" /> | ||
</Paper> */} | ||
<SocietyExec sid = {ecats.compsoc}/> | ||
<AlumniAccordions members={alumni.compsoc} sid = {ecats.compsoc} color="rgb(80 161 99)" /> |
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.
why are we still passing the members as a prop, remove it
<AlumniAccordions members={alumni.compsoc} sid = {ecats.compsoc} color="rgb(80 161 99)" /> | |
<AlumniAccordions sid = {ecats.compsoc} color="rgb(80 161 99)" /> |
@@ -0,0 +1,153 @@ | |||
import React 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.
This file is unnessary, please delete it
Earlier Execom and Alumni data is stored in local storage. This data is redirected to api. Changes are done in front-end code to pull the data from api.