Skip to content
This repository has been archived by the owner on Mar 15, 2018. It is now read-only.

Initial Variables required unnecesarily #57

Open
sebasgarcep opened this issue Feb 12, 2016 · 3 comments
Open

Initial Variables required unnecesarily #57

sebasgarcep opened this issue Feb 12, 2016 · 3 comments

Comments

@sebasgarcep
Copy link

First of all, nice project! I want to point out an issue I found recently. I tried implementing this in one of my projects but the Container wouldn't render so I ended up tracing the error to the componentWillMount lifecycle method of the Container, specifically to this line: https://github.com/RickWong/react-transmit/blob/master/src/lib/createContainer.js#L239. Going down that function call I noticed that this function returns false if the isRootContainer property of the Container is false, which in my case was because I hadn't declared initialVariables (https://github.com/RickWong/react-transmit/blob/master/src/lib/createContainer.js#L30). Once I set initialVariables to {} it worked as expected, but in my humble opinion that should have been uneccesary as I don't use any variables in my fragment functions. Or is this behaviour intended?

EDIT: Even if this behaviour is intended I don't think it should fail silently.

@mauriciopasquier
Copy link

I'll just quote from closed issue #49 where @RickWong said:

I really need to document this somewhere. Containers with initialVariables are so-called "root containers" and their fragments are fetched at start. Containers without initialVariables will never be automatically fetched. It's by design.

@RickWong
Copy link
Owner

It's really a lack of API design. I know. You guys have any suggestions to make it better?

@sebasgarcep
Copy link
Author

Console warnings (though I've been reading they aren't portable) and something in the docs might be useful. I won't be available for a couple of days so that is as much help as I can give.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants