Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Version 2 #66

Closed
wants to merge 12 commits into from
Closed

Version 2 #66

wants to merge 12 commits into from

Conversation

jaredpalmer
Copy link
Owner

@jaredpalmer jaredpalmer commented Jun 8, 2016

I've been thinking about this PR for a while now. Nothing changes functionality wise, it just refactors the src directory into client, common, and server. This is to ultimately make room for an upcoming directory called workers, which will hold microservices/Heroku workers that communicate with the main app via RabbitMQ.

Fixes #55

@jaredpalmer jaredpalmer temporarily deployed to spseo-pr-66 June 8, 2016 00:14 Inactive
@jaredpalmer jaredpalmer temporarily deployed to spseo-pr-66 June 8, 2016 00:19 Inactive
@jaredpalmer jaredpalmer temporarily deployed to spseo-pr-66 June 8, 2016 00:25 Inactive
@jaredpalmer jaredpalmer temporarily deployed to spseo-pr-66 June 8, 2016 00:40 Inactive
@jaredpalmer jaredpalmer temporarily deployed to spseo-pr-66 June 8, 2016 00:40 Inactive
This restore HMR. It also swaps the `indexRoute` of the application.
The About page is now the primary view. PostList is now at `/posts`.
@jaredpalmer jaredpalmer temporarily deployed to spseo-pr-66 June 8, 2016 01:29 Inactive
@jaredpalmer jaredpalmer temporarily deployed to spseo-pr-66 June 8, 2016 01:40 Inactive
@jaredpalmer jaredpalmer temporarily deployed to spseo-pr-66 June 8, 2016 02:00 Inactive
Middleware is pretty opinionated, in order to make things work I would
have needed to use a custom solution. This new code is agnostic with
respect to whether you use `isomorphic-fetch` or `axios` or whatevs.
@jaredpalmer jaredpalmer temporarily deployed to spseo-pr-66 June 8, 2016 20:53 Inactive
@jaredpalmer jaredpalmer temporarily deployed to spseo-pr-66 June 8, 2016 20:54 Inactive
* add demo selectors for reducers

* fix tests
@jaredpalmer jaredpalmer changed the title refactor Version 2 Jun 8, 2016
@jaredpalmer jaredpalmer closed this Jun 8, 2016
@jaredpalmer jaredpalmer deleted the tooling branch June 8, 2016 21:44
@jaredpalmer jaredpalmer temporarily deployed to spseo-pr-73 June 8, 2016 21:45 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant