search

pulilab / vue-people

VuePeople lists and connects Vue.JS developers around the world.
https://vuepeople.org
MIT License
172 stars 36 forks source link

Feature/user profile enhancements #27

Closed DonNicoJs closed 6 years ago

DonNicoJs commented 6 years ago

Several fixes / Enhancements: fixes #13 fixes #15 fixes #22 fixes #26

DonNicoJs commented 6 years ago

@nullcode still missing the logic to exclude the email from the api answers in the appropriate Views. I did not wanted to mess with it since I am not sure which is the real proper way to do so since is possible to do it with at least 5 different methods.

DonNicoJs commented 6 years ago

@nullcode small change on the email validator, I tested it and it works. I think is done, waiting for your approval

nullcode commented 6 years ago

@lordfuoco looks, good. Please test this scenario manually: set your email to hidden, logout-login, change your email If that works you can merge. Thanks

DonNicoJs commented 6 years ago

@nullcode scenarios tested, all working, removed unused import. I'll merge on develop branch