prepareVariables + routing

[devknoll]

If you pass in params from the route using e.g.

viewer: () => Relay.QL`query { viewer }`

then prepareVariables isn't called on the override parameters passed by the route (just initialVariables).

It seems like we should probably run the override parameters through too, so that everything going into the fragment query goes through prepareVariables first.

[devknoll]

Double checked the Relay source and debugged a little harder, and found out that prepareVariables is actually being called here, but some other strange thing is happening. I'll open a new issue when I discover it.

[josephsavona]

I'm working on a related change that should address this.
2015年9月3日(木) 14:10 Gerald Monaco notifications@github.com:

If you pass in params from the route using e.g.

viewer: () => Relay.QLquery { viewer }

then prepareVariables on the container is called, but prevVariables is an
empty object, rather than the params passed in from the route as would be
expected.

It seems like maybe we should also run any override parameters through
prepareVariables as well.

—
Reply to this email directly or view it on GitHub
#235.

[taion]

I do notice an odd thing here where the variables passed in from a wrapper seem to clobber initialVariables.