-
Notifications
You must be signed in to change notification settings - Fork 0
Issues: gazaskygeeks/mpcc-assistant
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
Author
Label
Projects
Milestones
Assignee
Sort
Issues list
Suggestion: move hostNewMentorModal / createNewMentorModal axios posts into action creators
#124
opened Apr 18, 2018 by
tomduggan85
MentorForm.submitForm() should pass state, not props.form, to action creator
#123
opened Apr 16, 2018 by
tomduggan85
Suggestion: Display "success" message to user after posting flight info
#122
opened Apr 16, 2018 by
tomduggan85
Suggestion: wrap submit-waiver input elements in form tag, it will be easier to get their values
#121
opened Apr 11, 2018 by
tomduggan85
Suggestion: Pull out individual notifications into their own component
#120
opened Apr 11, 2018 by
tomduggan85
Suggestion: split up the concept of a "Mentor" and a "Mentor visit"
#117
opened Apr 2, 2018 by
tomduggan85
You could get rid of the "mentors[0] &&" in the dashboard render code
#114
opened Mar 29, 2018 by
tomduggan85
You could pull the mentor card code out into its own component
#113
opened Mar 27, 2018 by
tomduggan85
Consider replacing navbar's create/update mentor Dropdown with some other html element
#112
opened Mar 27, 2018 by
tomduggan85
Navbar becomes a connected component in two different places
#111
opened Mar 27, 2018 by
tomduggan85
Auth / login flow to hide logged-in pages if user is not logged in
#110
opened Mar 23, 2018 by
tomduggan85
Navbar can check state for loggedIn instead of checking location
#109
opened Mar 23, 2018 by
tomduggan85
ProTip!
Type g i on any issue or pull request to go back to the issue listing page.