diff --git a/src/containers/SearchBar/actions.js b/src/containers/SearchBar/actions.js index c15f8068ae269e7622734cc861a02d4164320a8c..2db65a2ab7053390e1be125799dac87e84831784 100644 --- a/src/containers/SearchBar/actions.js +++ b/src/containers/SearchBar/actions.js @@ -2,9 +2,9 @@ import ReactGA from 'react-ga'; import { UPDATE_BUILDING_SEARCH_PARAMS, - LOAD_BUILDINGS, - LOAD_BUILDINGS_SUCCESS, - LOAD_BUILDINGS_ERROR, + SEARCH_BUILDINGS, + SEARCH_BUILDINGS_SUCCESS, + SEARCH_BUILDINGS_ERROR, } from './constants'; /** @@ -32,15 +32,15 @@ export function updateBuildingSearchParams(term, type, borough, zipcode, city) { * @returns {object} An action object with a type of LOAD_BUILDINGS * and filterName and value */ -export function loadBuildings(args) { +export function searchBuildings(args) { return { - type: LOAD_BUILDINGS, + type: SEARCH_BUILDINGS, args, }; } // TODO: Add doctring -export function buildingsLoaded(buildings, args) { +export function buildingsSearched(buildings, args) { if (args) { const address = `Address searched: ${args.address}`; if (buildings.length > 0) { @@ -50,16 +50,16 @@ export function buildingsLoaded(buildings, args) { } } return { - type: LOAD_BUILDINGS_SUCCESS, + type: SEARCH_BUILDINGS_SUCCESS, payload: buildings, }; } // TODO: Add doctring -export function buildingsLoadingError(error, args) { +export function buildingsSearchingError(error, args) { ReactGA.event({ category: 'Search', action: 'Error', label: `Searched for ${args.address}, Failed due to ${error}` }); return { - type: LOAD_BUILDINGS_ERROR, + type: SEARCH_BUILDINGS_ERROR, error, }; } diff --git a/src/containers/SearchBar/constants.js b/src/containers/SearchBar/constants.js index e5988943459ca368c010a33ff75b435a665a84a9..b0a679aade6039ffeee9046c19a158697d64d6a6 100644 --- a/src/containers/SearchBar/constants.js +++ b/src/containers/SearchBar/constants.js @@ -1,5 +1,5 @@ export const FETCH_BUILDINGS = 'FETCH_BUILDINGS'; export const UPDATE_BUILDING_SEARCH_PARAMS = 'UPDATE_BUILDING_SEARCH_PARAMS'; -export const LOAD_BUILDINGS = 'LOAD_BUILDINGS'; -export const LOAD_BUILDINGS_SUCCESS = 'LOAD_BUILDINGS_SUCCESS'; -export const LOAD_BUILDINGS_ERROR = 'LOAD_BUILDINGS_ERROR'; +export const SEARCH_BUILDINGS = 'SEARCH_BUILDINGS'; +export const SEARCH_BUILDINGS_SUCCESS = 'SEARCH_BUILDINGS_SUCCESS'; +export const SEARCH_BUILDINGS_ERROR = 'SEARCH_BUILDINGS_ERROR'; diff --git a/src/containers/SearchBar/index.js b/src/containers/SearchBar/index.js index 1ef6ff9234279d6fdbb3f341ef079d05df7894f2..ee34c163ef653fd321608b4670b45ce59190eac1 100644 --- a/src/containers/SearchBar/index.js +++ b/src/containers/SearchBar/index.js @@ -9,8 +9,8 @@ import { connect } from 'react-redux'; import { bindActionCreators } from 'redux'; import { updateBuildingSearchParams, - loadBuildings, - buildingsLoaded, + searchBuildings, + buildingsSearched, } from './actions'; import './styles.css'; import { buildingListPropTypes } from './propTypes'; @@ -58,7 +58,7 @@ class SearchBar extends Component { // Use the buildings stored in cookies to populate the buildingList const buildingList = localStorage.activeBuildings; if (buildingList !== undefined) { - this.props.buildingsLoaded(JSON.parse(buildingList)); + this.props.buildingsSearched(JSON.parse(buildingList)); } } } @@ -117,7 +117,7 @@ class SearchBar extends Component { if (stateSource.type === 'address + borough' && stateSource.borough) { args.boro = SEARCH_BOROUGH_LIST[stateSource.borough]; } - this.props.loadBuildings(args); + this.props.searchBuildings(args); } checkTermLength = () => { @@ -350,16 +350,16 @@ class SearchBar extends Component { SearchBar.propTypes = { buildingList: buildingListPropTypes, updateBuildingSearchParams: PropTypes.func, - loadBuildings: PropTypes.func, - buildingsLoaded: PropTypes.func, + searchBuildings: PropTypes.func, + buildingsSearched: PropTypes.func, setNavbar: PropTypes.func, }; function mapDispatchToProps(dispatch) { return bindActionCreators({ updateBuildingSearchParams, - loadBuildings, - buildingsLoaded, + searchBuildings, + buildingsSearched, }, dispatch); } diff --git a/src/containers/SearchBar/reducer.js b/src/containers/SearchBar/reducer.js index 7ba9238db131226cdbf651ef6b7a67b14d16fc84..dc5aebc2313ea4870de6d9c9d7073a4730b9b0a7 100644 --- a/src/containers/SearchBar/reducer.js +++ b/src/containers/SearchBar/reducer.js @@ -1,8 +1,8 @@ import { UPDATE_BUILDING_SEARCH_PARAMS, - LOAD_BUILDINGS, - LOAD_BUILDINGS_SUCCESS, - LOAD_BUILDINGS_ERROR, + SEARCH_BUILDINGS, + SEARCH_BUILDINGS_SUCCESS, + SEARCH_BUILDINGS_ERROR, } from './constants'; const initState = { @@ -27,20 +27,20 @@ export default function (state = initState, action) { zipcode: action.zipcode, city: action.city, }; - case LOAD_BUILDINGS: + case SEARCH_BUILDINGS: return { ...state, loading: true, error: false, }; - case LOAD_BUILDINGS_SUCCESS: + case SEARCH_BUILDINGS_SUCCESS: return { ...state, buildings: action.payload, loading: false, error: false, }; - case LOAD_BUILDINGS_ERROR: + case SEARCH_BUILDINGS_ERROR: return { ...state, loading: false, diff --git a/src/containers/SearchBar/sagas.js b/src/containers/SearchBar/sagas.js index cf49827fd247353997a2757029240027fa8bc4ce..2b9a5c3b9d96d2ccb110920b276cda99a2d234eb 100644 --- a/src/containers/SearchBar/sagas.js +++ b/src/containers/SearchBar/sagas.js @@ -3,12 +3,12 @@ import request from '../../utils/request'; import { getHeaders, buildingsURL } from '../../utils/restServices'; import { - LOAD_BUILDINGS, + SEARCH_BUILDINGS, } from './constants'; import { - buildingsLoaded, - buildingsLoadingError, + buildingsSearched, + buildingsSearchingError, } from './actions'; /** @@ -28,9 +28,9 @@ function* getBuildingsByFilter(action) { ); console.log(res); // eslint-disable-line if (!res.err) { - yield put(buildingsLoaded(res.data, action.args)); + yield put(buildingsSearched(res.data, action.args)); } else { - yield put(buildingsLoadingError(res.err, action.args)); + yield put(buildingsSearchingError(res.err, action.args)); } } @@ -38,5 +38,5 @@ function* getBuildingsByFilter(action) { * TODO: docstring */ export default function* buildingsWatcher() { - yield takeLatest(LOAD_BUILDINGS, getBuildingsByFilter); + yield takeLatest(SEARCH_BUILDINGS, getBuildingsByFilter); } diff --git a/src/screens/BuildingsHomePage/index.js b/src/screens/BuildingsHomePage/index.js index 89cb4f50b36f95fc9bc1ee2ccc38657ada79c725..9e4b4eae85acd5e2f1cc35c26367f542f5a48e58 100644 --- a/src/screens/BuildingsHomePage/index.js +++ b/src/screens/BuildingsHomePage/index.js @@ -9,8 +9,8 @@ import BuildingListTable from '../../components/BuildingListTable'; import { buildingListPropTypes } from '../../containers/SearchBar/propTypes'; import { updateBuildingSearchParams, - loadBuildings, - buildingsLoaded, + searchBuildings, + buildingsSearched, } from '../../containers/SearchBar/actions'; import userPropType from '../../containers/User/propTypes'; import NotAuthorized from '../../components/NotAuthorized'; @@ -29,7 +29,7 @@ class BuildingsHomePage extends Component { logoOnClick={() => { const buildingList = localStorage.activeBuildings; if (buildingList !== undefined) { - this.props.buildingsLoaded(JSON.parse(buildingList)); + this.props.buildingsSearched(JSON.parse(buildingList)); } }} user={this.props.user} @@ -45,7 +45,7 @@ class BuildingsHomePage extends Component { buildings={this.props.buildingList.buildings} loading={this.props.buildingList.loading} updateSearchParams={this.props.updateBuildingSearchParams} - loadBuildings={this.props.loadBuildings} + loadBuildings={this.props.searchBuildings} error={this.props.buildingList.error} /> @@ -57,8 +57,8 @@ class BuildingsHomePage extends Component { BuildingsHomePage.propTypes = { buildingList: buildingListPropTypes, updateBuildingSearchParams: PropTypes.func, - loadBuildings: PropTypes.func, - buildingsLoaded: PropTypes.func, + searchBuildings: PropTypes.func, + buildingsSearched: PropTypes.func, user: userPropType, }; @@ -68,8 +68,8 @@ function mapStateToProps({ buildingList }) { function mapDispatchToProps(dispatch) { return bindActionCreators({ updateBuildingSearchParams, - loadBuildings, - buildingsLoaded, + searchBuildings, + buildingsSearched, }, dispatch); }