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

fiixes #109 - fixed account apps #115

Merged
merged 1 commit into from
Mar 13, 2017
Merged

Conversation

djmgit
Copy link
Member

@djmgit djmgit commented Mar 1, 2017

fixed the following apps:

signup app
login app
register public key app
password recovery app
view permissions app
reset password app(not tested)

fixed the following  apps:

signup app
login app
register public key app
password recovery app
view permissions app
reset password app(not tested)
@djmgit
Copy link
Member Author

djmgit commented Mar 1, 2017

demo link : http:https://loklakapps.surge.sh/

@djmgit
Copy link
Member Author

djmgit commented Mar 1, 2017

the password reset app is not tested as I am not sure how to generate token.
However I have fixed the api calling by using jsonp

@yukiisbored
Copy link
Member

Typo on commit message, please change it and pleae do test the reset password app, you can test it by running a local instance of loklak or request someone else to test it. Don't put the "not tested" on the commit message.

@djmgit
Copy link
Member Author

djmgit commented Mar 4, 2017

@yukiisbored I have a local instance of the server running. I tried calling the api on the local server, but its giving error : token not specified .

@mariobehling
Copy link
Member

Let's merge and iterate.

@djmgit Please open an issue for the remaining.

@mariobehling mariobehling merged commit 1adb670 into loklak:master Mar 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants