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

Add user email into the response #100

Merged
merged 2 commits into from
Jul 26, 2016
Merged

Add user email into the response #100

merged 2 commits into from
Jul 26, 2016

Conversation

mitogh
Copy link
Member

@mitogh mitogh commented Jul 25, 2016

  • What kind of change does this PR introduce? (Bug fix, feature,
    docs update, ...)

Feature

  • What is the current behavior? (You can also link to an open issue
    here)

There is no user email on the controller

  • What is the new behavior (if this is a feature change)?

Adds the user email in the controller

  • Does this PR introduce a breaking change? (What changes might
    users need to make in their application due to this PR?)

No

  • Other information:

@mitogh
Copy link
Member Author

mitogh commented Jul 25, 2016

@andybar2 or @adamf321 would you mind taking a look at this one?

Thank you.

@andybar2 andybar2 merged commit a90d2e7 into master Jul 26, 2016
@andybar2 andybar2 deleted the add-user-email branch July 26, 2016 13:18
@andybar2
Copy link
Collaborator

Ready Cris! I also published ln-patternlab@0.4.0 with your changes.

2016-07-25 20:56 GMT-03:00 Crisoforo Gaspar Hernández <
notifications@github.com>:

@andybar2 https://github.com/andybar2 or @adamf321
https://github.com/adamf321 would you mind taking a look at this one?

Thank you.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#100 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AH38XET8l77DinMCCy0D9lbl_-0HBUWsks5qZU0qgaJpZM4JUqOB
.

@mitogh
Copy link
Member Author

mitogh commented Jul 26, 2016

Thank you

On Jul 26, 2016, at 8:22 AM, Andres Barrios notifications@github.com wrote:

Ready Cris! I also published ln-patternlab@0.4.0 with your changes.

2016-07-25 20:56 GMT-03:00 Crisoforo Gaspar Hernández <
notifications@github.com>:

@andybar2 https://github.com/andybar2 or @adamf321
https://github.com/adamf321 would you mind taking a look at this one?

Thank you.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#100 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AH38XET8l77DinMCCy0D9lbl_-0HBUWsks5qZU0qgaJpZM4JUqOB
.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants