Skip to content

hot reload + api / production on 3000 to match CRA#1

Open
kdmcclel wants to merge 8 commits into
ayroblu:masterfrom
kdmcclel:master
Open

hot reload + api / production on 3000 to match CRA#1
kdmcclel wants to merge 8 commits into
ayroblu:masterfrom
kdmcclel:master

Conversation

@kdmcclel

@kdmcclel kdmcclel commented May 1, 2017

Copy link
Copy Markdown

I liked your approach to avoiding an eject while adding SSR / API server. I wanted to enhance development by getting the API running beside the CRA dev server. Also added in hot reloading for containers / reducers.

After I got that working for my project I thought I'd make the changes to yours and submit a pull request for anyone who finds this moving forward.

I ended up fleshing out your api enough to confirm the api server responds during development. When you go to the Second Page it fetches /api/users/1 and updates the store. Returning to the first page will show the new email address. It's pretty bare bones.

@ayroblu

ayroblu commented May 6, 2017

Copy link
Copy Markdown
Owner

Okay, so made some changes, here's my thoughts (didn't test my changes)
Proxy - cool didn't know / forgot about that one
concurrently - I actually like keeping them seperate, but that's also cool
Added api endpoint - I guess sure
Using the api - I do it a different way, so I've done it my preferred way
ApiUrl - don't need this with the proxy
Sagas - ewww so much code, but sure, I'm not sure about the syntax so I'm gonna assume that's fine
Hot reload - didn't know about this, that's cool

Once changes are tested I'll merge in

@kdmcclel

kdmcclel commented May 6, 2017

Copy link
Copy Markdown
Author

Cool looks good. Yeah I started with just concurrently / hot reloading / proxy then thought I ought to make sure the API proxy was working before submitting. Tried to follow what you had started with the commented out saga stuff.

I wasn't sure res.json() returned status at all or just the data so the saga code may have to change now.

@ayroblu

ayroblu commented May 8, 2017

Copy link
Copy Markdown
Owner

Yeah, I haven't gotten to the point where I actually need sagas. I leave the code there in case its needed cause it's common practice, but it's such a pain and 99% unnecessary

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.

2 participants