Skip to content

refactor(model:user): exclude password info by default#1049

Closed
Awk34 wants to merge 1 commit intocanaryfrom
refactor-user-model-password-hiding
Closed

refactor(model:user): exclude password info by default#1049
Awk34 wants to merge 1 commit intocanaryfrom
refactor-user-model-password-hiding

Conversation

@Awk34
Copy link
Copy Markdown
Collaborator

@Awk34 Awk34 commented Jul 9, 2015

thanks to guiltry for the original commit (89980c5)

thanks to guiltry for the original commit (89980c5)
This was referenced Jul 9, 2015
@Awk34 Awk34 added this to the 2.3.0 milestone Jul 15, 2015
@kingcody
Copy link
Copy Markdown
Member

Whats the status on this one?

@Awk34
Copy link
Copy Markdown
Collaborator Author

Awk34 commented Jul 16, 2015

The tests fail, I haven't looked into fixing them yet

@kingcody
Copy link
Copy Markdown
Member

So without additional plugins, I'm not sure we can get this to work for sequelize. For now it may just need to be a mongoose feature.

@Awk34 Awk34 closed this Feb 9, 2016
@Awk34 Awk34 deleted the refactor-user-model-password-hiding branch June 21, 2016 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants