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

Id site #37

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from
Open

Id site #37

wants to merge 8 commits into from

Conversation

avojnovicDk
Copy link
Contributor

Here are changes for the #19 issue. I reverted STORMPATH_APPLICATION setting change and added docstrings and tests for ID Site feature.

denibertovic and others added 8 commits December 3, 2014 15:53
this fixes the scenario when the web app blows up if
STORMPATH_APPLICAITON is a href.
- Added a setting STORMPATH_ENABLE_ID_SITE which basically switches the
built in manual views/forms for the ID site ones. Another settings called
STORMPATH_ID_SITE_CALLBACK_URL was added as well.

- Updated the stormpath sdk dependency to point to the latest version.
I was hitting the search issue with '/' and '&' charaters
resulting in a bad error message informing me of invalid API
credentials, and this has been resolved in the latest sdk version.

- Added quickstart docs for this feature
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.

2 participants