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

support for lists #5

Open
yeedle opened this issue Apr 5, 2019 · 2 comments
Open

support for lists #5

yeedle opened this issue Apr 5, 2019 · 2 comments

Comments

@yeedle
Copy link

yeedle commented Apr 5, 2019

The javascript json-mask implementation supports lists/arrays, while this implementation doesn't.

json-mask:

> const mask = require('json-mask');
> mask({ p: [{a: 1, b: 2 }], z: 1 }, 'p/a,z')
{ p: [ { a: 1 } ], z: 1 }

jsonmask:

>>> import jsonmask
>>> jsonmask.apply_json_mask({'p': [{'a': 1, 'b': 2}], 'z': 1}, 'p/a,z')
{'p': [{'a': 1, 'b': 2}], 'z': 1}

It would be nice if jsonmask would support this. If you'd like I can open a PR.

@harmonm
Copy link

harmonm commented Apr 30, 2019

@yeedle i need this too

@juanyque
Copy link

Hello @yeedle and @harmonm,

Just to tell you that: after trying to push improvements in this library without success, I have decided to clone it and evolve it on my own, now there is a new library in PyPI, jsonmask-ng, with the proposed improvements and others that I have needed.

I'm doing my best, but I'm no so much expert in python. Any comment about it will be welcome and I will try to attend suggestions.

Regards

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