Skip to content
This repository has been archived by the owner on Jun 24, 2020. It is now read-only.

Initial React refactor #434

Merged
merged 12 commits into from
May 18, 2015
Merged

Initial React refactor #434

merged 12 commits into from
May 18, 2015

Conversation

virginiacc
Copy link
Contributor

Changes

  • Add react & flux
  • JS refactor
  • Input table html/css refactor

Notes

  • Mobile functionality needs to be restored
  • CSS updates/refactor needed

Review

@cfarm

@cfarm
Copy link
Contributor

cfarm commented May 14, 2015

👍 lookin good. do we want to merge this before or after #432? i have no preference

@cfarm cfarm mentioned this pull request May 15, 2015
4 tasks
cfarm added a commit that referenced this pull request May 18, 2015
@cfarm cfarm merged commit adcc555 into cfpb:master May 18, 2015
@ascott1
Copy link
Member

ascott1 commented May 19, 2015

If y'all refactor the rate-checker to use React, can I help? Pretty please?

@amymok
Copy link
Contributor

amymok commented May 20, 2015

@ascott1 maybe we can also pair :) i need help with React, so confused :)

@cfarm
Copy link
Contributor

cfarm commented May 20, 2015

@ascott1 I don't think that's on our radar butttt you can help us with LC :)

@ascott1
Copy link
Member

ascott1 commented May 20, 2015

@ascott1 maybe we can also pair :) i need help with React, so confused :)

Sounds fun! I'd love to

@ascott1 I don't think that's on our radar butttt you can help us with LC :)

🤘

@cfarm cfarm mentioned this pull request May 22, 2015
4 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants