search

ubclaunchpad / Internado

🌪 Find Internship Opportunities
3 stars 1 forks source link

Fixed Issue with Pressing Enter When Searching for Jobs #153

Closed cormacjmollitor closed 5 years ago

cormacjmollitor commented 5 years ago

Pressing enter in the search bar now calls the backend and populates the results table. Closes #144

cormacjmollitor commented 5 years ago

@cowmanjoe We're probably not going to use Redux. This commit actually doesn't use Redux but uses a global store file store.js instead. I made a change to rename the Redux store file to reduxStore.js because it was named store.js before. If we're set on abandoning Redux, we can delete the reduxStore.js file and remove the imports for it.

cowmanjoe commented 5 years ago

Sorry I think I may have misunderstood your comment.

cowmanjoe commented 5 years ago

I see now. You renamed the store to reduxStore, but haven't deleted it yet because we're not sure about whether we're using it or not. Is that correct?

cormacjmollitor commented 5 years ago

All good. Yeah, I just renamed it to reduxStore because even if we abandon Redux, we'll probably still want a global store file so I made a new one called store.js

cormacjmollitor commented 5 years ago

I agree. I'll get rid of the redux store file and the imports for it before merging.