Skip to content

Conversation

@evansiroky
Copy link
Contributor

  • Bug fix for checking if query is valid
  • Adding ability to have customizable otp query function
  • Add redux action for switching locations

also add test case for this function
This option allows setting the custom builder function in the config
when creating the reducer’s initial state.  When using all of the
reducers together they make calls to the planTrip action, but there
wasn’t really a way to pass in a custom function that way.  For now it
seems like the only way to do this could be to set something in the
config of the state.

This PR includes the work done in #12.
* Add Switch Button to lib
* Alphabetize all the things in lib/index.js
* rename file for switch button
@codecov-io
Copy link

codecov-io commented Mar 23, 2017

Codecov Report

Merging #15 into master will decrease coverage by 0.07%.
The diff coverage is 5.88%.

@@            Coverage Diff            @@
##           master     #15      +/-   ##
=========================================
- Coverage     7.2%   7.13%   -0.08%     
=========================================
  Files          34      35       +1     
  Lines         555     575      +20     
=========================================
+ Hits           40      41       +1     
- Misses        515     534      +19
Impacted Files Coverage Δ
lib/util/state.js 100% <ø> (+14.28%) ⬆️
lib/index.js 0% <0%> (ø) ⬆️
lib/components/form/switch-button.js 0% <0%> (ø)
lib/actions/map.js 0% <0%> (ø) ⬆️
lib/actions/api.js 86.36% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f162be7...53393d9. Read the comment docs.

@evansiroky evansiroky requested a review from landonreed March 23, 2017 15:47
@evansiroky evansiroky merged commit 07f2631 into master Mar 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants