-
Notifications
You must be signed in to change notification settings - Fork 3
feat(Donor setup after sign up): [FAIR-1] #23
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
base: main
Are you sure you want to change the base?
Conversation
… began creating the set up page for PROGRAM DIRECTORS
…ector and then they are directed to choose their organization as a user however you must have an admin account to currently do this
…setting up. Can be used for the non-profit user as well
francisli
left a comment
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.
Donor related CRUD seems to be working well, but the Student form is a bit broken, and NonProfitPartner seems still unfinished...
client/src/Students/StudentsForm.js
Outdated
| </div> | ||
| <div className="mb-3"> | ||
| <label className="form-label">Student Comfirmed?</label> | ||
| <input className="form-control" type="text" name="Confirm" value={anStudent.confirm} onChange={onChange} /> |
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.
Editing the form is not working...
Make sure that the name attribute and the variable name in the student object match EXACTLY for the code to work... you see they don't match between userEducation/userEdu and Confirm/confirm...
They should match exactly how they are named in the Student model.
| import { useHistory, useParams} from "react-router-dom"; | ||
| import Api from "../Api"; | ||
|
|
||
| function StudentsForm() { |
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 name of this file and component should really be StudentForm (singular Student)... this form edits ONE student, not multiple...
… new title for website and attempted to change favicon.
cesar-ca
left a comment
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 branch is looking great! Amazing job
[Do not merge yet]
cesar-ca
left a comment
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.
LG
… andres-donor-branch-3
…n/techequity into andres-donor-branch-3
No description provided.