Closed chimucong closed 7 years ago
Thanks for taking the time to create a pull request!
I like the idea and see the usefulness. Do you have time to add some test cases as well?
I'll merge your PR, but I'd like to get some tests and documentation around the feature too.
On Thu, Sep 22, 2016 at 12:00 AM, chimucong notifications@github.com wrote:
I have to define record like this sometimes
-record(user, {name, age, password}).
and I don't want to show password in JSON.
I just added excluded_field to aeon so that fields with excluded_field won't show in JSON.
Hope you like it 😂
You can view, comment on, or merge this pull request online at:
https://github.com/garret-smith/aeon/pull/1 Commit Summary
- add excluded_field
File Changes
- M src/aeon.erl https://github.com/garret-smith/aeon/pull/1/files#diff-0 (2)
- M src/aeon_common.erl https://github.com/garret-smith/aeon/pull/1/files#diff-1 (8)
- M src/aeon_to_erl.erl https://github.com/garret-smith/aeon/pull/1/files#diff-2 (55)
- M src/aeon_to_jsx.erl https://github.com/garret-smith/aeon/pull/1/files#diff-3 (29)
Patch Links:
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/garret-smith/aeon/pull/1, or mute the thread https://github.com/notifications/unsubscribe-auth/AARTVSaNyk1O6aX04WpeDfkKkytnMNqfks5qsielgaJpZM4KDlUr .
I have to define record like this sometimes
and I don't want to show
password
in JSON.I just added
excluded_field
toaeon
so that fields withexcluded_field
won't show in JSON.Hope you like it :joy: