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

Fixed JSON bug and made a short explanation about API #3

Merged
merged 4 commits into from Aug 6, 2012
Merged

Fixed JSON bug and made a short explanation about API #3

merged 4 commits into from Aug 6, 2012

Conversation

@ghost
Copy link

@ghost ghost commented Aug 5, 2012

Hi!

Somebody on Reddit was talking about this project, and they wanted an API for integration with Powershell. I initially wanted to add that functionality, but I saw an API-controller that didn't do much, and saw an opportunity to use the PasswordController for it instead.

Let me know what you think.

Greetings,
Kasper.

@pglombardo
Copy link
Owner

@pglombardo pglombardo commented Aug 6, 2012

Hi Kasper - this looks great/thanks!

One request though, could you remove the .rvmrc from the pull request? I think you added that by mistake.

@ghost
Copy link
Author

@ghost ghost commented Aug 6, 2012

Oh, I did a git add -A, and it also added the .rvmrc, but it is removed now :-)

@pglombardo
Copy link
Owner

@pglombardo pglombardo commented Aug 6, 2012

Awesome thanks! I'll update pwpush.com before EOD today.

pglombardo pushed a commit that referenced this pull request Aug 6, 2012
Fixed JSON bug and made a short explanation about API
@pglombardo pglombardo merged commit c2bd7e6 into pglombardo:master Aug 6, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants