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

toJS() doesn't seem to recursively work? #26

Open
krainboltgreene opened this issue Apr 2, 2016 · 3 comments
Open

toJS() doesn't seem to recursively work? #26

krainboltgreene opened this issue Apr 2, 2016 · 3 comments

Comments

@krainboltgreene
Copy link
Collaborator

Before (With console.dir()):

{ name: 'lacqueristas',
  request:
   { method: [Getter/Setter],
     url: [Getter/Setter],
     version: [Getter/Setter],
     headers: [Getter/Setter],
     body: [Getter/Setter] },
  response: { status: [Getter/Setter], headers: [Getter/Setter] }

After:

{ name: 'lacqueristas',
  request:
   { method: 'OPTIONS',
     url: '/v1/activities?include=actor&page%5Bnumber%5D=0&page%5Bsize%5D=10',
     version: { major: [Getter/Setter], minor: [Getter/Setter] },
     headers:
      { host: [Getter/Setter],
        connection: [Getter/Setter],
        'access-control-request-method': [Getter/Setter],
        origin: [Getter/Setter],
        'user-agent': [Getter/Setter],
        'access-control-request-headers': [Getter/Setter],
        accept: [Getter/Setter],
        dnt: [Getter/Setter],
        referer: [Getter/Setter],
        'accept-encoding': [Getter/Setter],
        'accept-language': [Getter/Setter] },
     body: '' },
  response:
   { method: 'OPTIONS',
     url: '/v1/activities?include=actor&page%5Bnumber%5D=0&page%5Bsize%5D=10',
     version: { major: [Getter/Setter], minor: [Getter/Setter] },
     headers:
      { host: [Getter/Setter],
        connection: [Getter/Setter],
        'access-control-request-method': [Getter/Setter],
        origin: [Getter/Setter],
        'user-agent': [Getter/Setter],
        'access-control-request-headers': [Getter/Setter],
        accept: [Getter/Setter],
        dnt: [Getter/Setter],
        referer: [Getter/Setter],
        'accept-encoding': [Getter/Setter],
        'accept-language': [Getter/Setter] },
     body: '' }
}

Without immu I can see the entire tree.

@scottcorgan
Copy link
Owner

This shouldn't be the case. It's lazily evaluated. Do you specific steps to reproduce?

@iamnewspecies
Copy link

I also have similar issue. For one key it is working and other key it is not working. For now I am using JSON.parse(JSON.stringify(data)). I hope this should not create any issue.

@scottcorgan
Copy link
Owner

@iamnewspecies can you post example code to reproduce the bug so I can try to fix it?

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

No branches or pull requests

3 participants