Skip to content
This repository has been archived by the owner on Apr 20, 2018. It is now read-only.

adding ability to remove from result.data object #64

Merged
merged 1 commit into from
Mar 30, 2016
Merged

Conversation

ekryski
Copy link
Member

@ekryski ekryski commented Mar 30, 2016

This is a workaround for #63 for auth. Adding a scope option would require more work.

@marshallswain
Copy link
Member

👍 :shipit:

@daffl
Copy link
Member

daffl commented Mar 30, 2016

Wy not just allow nested paths like .after({ get: remove('user.password') })?

@marshallswain
Copy link
Member

That's what I was thinking.

@ekryski
Copy link
Member Author

ekryski commented Mar 30, 2016

Ya I think that is the best solution. Do you guys have an idea on how best to implement it? I'd like to ship this as is and come back to it in #63. This is holding up auth 0.7, which is holding up the docs and generator updates.

@ekryski ekryski merged commit 73ad049 into master Mar 30, 2016
@ekryski ekryski deleted the remove-from-data branch March 30, 2016 20:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants