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

[$30] add provider name to provider login at top of banner #37

Open
kbowerma opened this issue Jan 6, 2017 · 4 comments
Open

[$30] add provider name to provider login at top of banner #37

kbowerma opened this issue Jan 6, 2017 · 4 comments

Comments

@kbowerma
Copy link
Contributor

kbowerma commented Jan 6, 2017

When you are logged in as a provider lets add the provider name underneath the users first and last name.

screen shot 2017-01-06 at 12 39 03 pm

@muzehyun
Copy link
Collaborator

muzehyun commented Jan 7, 2017

@kbowerma Can't reassign. Could you add me to this project so I can assign this to me?

@kbowerma
Copy link
Contributor Author

kbowerma commented Jan 7, 2017

@muzehyun you are now a collaborator and can assign issues to yourself.

@muzehyun
Copy link
Collaborator

muzehyun commented Jan 9, 2017

In current backend, provider is removed before returning as login result.
Can I change them to include provider info?
I need backend repository permission.

UserSchema.options.toObject.transform = function (doc, ret, options) { // eslint-disable-line no-unused-vars
  const sanitized = _.omit(ret, '__v', '_id', 'password', 'provider', 'createdAt', 'updatedAt');
  sanitized.id = doc._id;
  return sanitized;
};

@kbowerma
Copy link
Contributor Author

kbowerma commented Jan 9, 2017

thanks @muzehyun you have been added you can go ahead and update the response. add dsp-frontend/37 to your commit so we can track across repos

muzehyun added a commit to muzehyun/dsp-frontend that referenced this issue Jan 9, 2017
muzehyun added a commit to muzehyun/dsp-server that referenced this issue Jan 9, 2017
gondzo added a commit to topcoderinc/dsp-server that referenced this issue Jan 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants