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

[MNO-287] Developer Section in My Account #153

Merged
merged 3 commits into from
Nov 24, 2016

Conversation

hedudelgado
Copy link
Contributor

@hedudelgado hedudelgado force-pushed the MNO-287-developer-section branch 2 times, most recently from a58c766 to 5d3d96d Compare November 15, 2016 23:36
@hedudelgado hedudelgado changed the title [MNO-287] Developer Section in My Accoun [MNO-287] Developer Section in My Account Nov 15, 2016
@hedudelgado hedudelgado force-pushed the MNO-287-developer-section branch from 5d3d96d to 1c505d6 Compare November 15, 2016 23:41
@hedudelgado
Copy link
Contributor Author

@hedudelgado
Copy link
Contributor Author

@hedudelgado hedudelgado force-pushed the MNO-287-developer-section branch from 191be07 to 6a98602 Compare November 16, 2016 10:46
@alexnoox
Copy link
Contributor

@BrunoChauvet Please review

Frontend: maestrano/mno-enterprise-angular#93

@@ -34,6 +33,17 @@ def update
end
end

# PUT /mnoe/jpi/v1/current_user/register_developer
def register_developer
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please write a spec for this method

@hedudelgado hedudelgado force-pushed the MNO-287-developer-section branch 2 times, most recently from 8c23216 to 194dc64 Compare November 24, 2016 19:55
@hedudelgado hedudelgado force-pushed the MNO-287-developer-section branch from ccbcbaf to bdd4d81 Compare November 24, 2016 20:44
@hedudelgado
Copy link
Contributor Author

@BrunoChauvet

@alexnoox alexnoox merged commit 83b620d into maestrano:master Nov 24, 2016
aluqueGH pushed a commit to aluqueGH/mno-enterprise that referenced this pull request Jul 10, 2018
…_bug_fix

Added explanation when no reviews and added some style
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.

4 participants