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

Rename comments and strings from dummy dev to prompted auth #32

Merged
merged 1 commit into from
Sep 5, 2018

Conversation

shankari
Copy link
Contributor

@shankari shankari commented Sep 5, 2018

Only current use of dummy dev is in the fallback if a prompt is not specified
for prompted auth. This allows development to proceed without any
configuration, but removes all other references to obsolete code.

$ grep -r DummyDev src/
$ grep -r dummy-dev src/
$ grep -r Dummy src/
src//ios/PromptedAuth.m:        configPrompt = @"Dummy dev mode: Enter email";
src//android/PromptedAuth.java:            configPrompt = "Dummy dev mode: Enter email";
$ grep -r dummy src/
$

Only current use of dummy dev is in the fallback if a prompt is not specified
for prompted auth. This allows development to proceed without any
configuration, but removes all other references to obsolete code.

```
$ grep -r DummyDev src/
$ grep -r dummy-dev src/
$ grep -r Dummy src/
src//ios/PromptedAuth.m:        configPrompt = @"Dummy dev mode: Enter email";
src//android/PromptedAuth.java:            configPrompt = "Dummy dev mode: Enter email";
$ grep -r dummy src/
$
```
@shankari shankari merged commit a27cf38 into e-mission:master Sep 5, 2018
shankari added a commit to shankari/e-mission-phone that referenced this pull request Sep 20, 2018
- add a sample for `openid`
- add a sample for `token_list`
- update all samples from `dummy-dev` to `prompted-auth`
(consistent with e-mission/cordova-jwt-auth#28 and
e-mission/cordova-jwt-auth#32)

Update README to clarify + point to revised docs
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.

1 participant