Skip to content
This repository has been archived by the owner on Mar 7, 2019. It is now read-only.

Add Dev Branch to Master #41

Open
wants to merge 50 commits into
base: master
Choose a base branch
from
Open

Add Dev Branch to Master #41

wants to merge 50 commits into from

Conversation

billmwong
Copy link
Member

@billmwong billmwong commented May 26, 2017

This change is Reviewable

rsiegel812 and others added 30 commits April 30, 2017 22:39
…/update-plan-button

Conflicts:
	src/containers/SemesterListContainer.jsx
…/update-plan-button

Conflicts:
	src/reducers/LoginReducer.jsx
Conflicts:
	src/actions/actions.jsx
	src/reducers/ProgressTrackerReducer.jsx
Hotfix registration to add sane default name
Jeremy Garcia and others added 18 commits May 2, 2017 17:21
The login page was crashing on incorrect registers, and also
had no way of surfacing error messages. This commit fixes
the former problem and adds the state changes necessary
to display the latter.
The connect function wasn't quite woring right, which was causing ugly
console errors and incorrect rehydrating. This commit fixes that by
adding mapDispatchToProps() and using it right.
@billmwong billmwong added the wip label May 26, 2017
Conflicts:
	routes/index.js
	src/actions/actions.jsx
	src/reducers/LoginReducer.jsx
@billmwong
Copy link
Member Author

billmwong commented May 26, 2017

Known features in here:

  • Consolidated reducers with combineReducers() into [Student, Major, Courses, Progress, CoursePlanner]
  • Styled Calendar Page
  • "Update Plan" button to save plan to database
  • Register new user hotfix

TODO:

  • Check that this branch is functional
  • Check that register hotfix to make sure that works
  • Check that this code is good
  • Update any code as necessary

TODO from JIRA:

  • Make linked reqs talk to each other (e.g. When you select "Linearity I" for your "Linearity I" requirement, it should also chunk that into your "Math" credit requirement)
  • Pull the calendar all together
  • Sync client state to server
  • link to calendar page

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants