-
Notifications
You must be signed in to change notification settings - Fork 255
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
using several stores / overrides - merges not as expected #15
Comments
Does #13 work as you'd expect it? |
going over #13's diff, i don't see any changes to merge logic, so i'm not sure. i guess i'll have to use the commiter's branch and try it. |
@jondot hmm, that is how I would expect this to work as well. that is certainly the impression that I get from the usage of "hierarchical" in the README |
Here's another example: |
so in that case the movie param is undefined. is this intended? |
@jondot No. |
I'm investigating this. It will be fixed in the |
This is confirmed fixed in the |
The tests which confirm this are the gist from @jesusabdullah verbatim with some vows and assertions in 38fcbc7 |
This is fixed in |
Hi,
I'm trying to set up store as such:
I then expect the user values to override the global values such that:
but in every case i'm getting port:80 being resloved.
if i reverse the order like so:
then i get undefined values, because global declares more values than in user.
db is undefined -- seems to be overriden in user - but i'm not specifying it there in any case and expecting it to be merged from global.
anything i'm doing wrong here?
Thanks!
The text was updated successfully, but these errors were encountered: