-
Notifications
You must be signed in to change notification settings - Fork 992
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
[Feature ✨] - Create Password #459
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
app/src/main/AndroidManifest.xml
Outdated
<meta-data | ||
android:name="com.google.android.gms.wallet.api.enabled" | ||
android:value="true" /> | ||
|
||
<activity android:name=".ui.activities.CreatePasswordActivity"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's alphabetize this and use the settings transitions
@@ -74,6 +74,11 @@ class LoginActivity : BaseActivity<LoginViewModel.ViewModel>() { | |||
.compose(observeForUI()) | |||
.subscribe { showConfirmationSnackbar(login_toolbar, R.string.Got_it_your_changes_have_been_saved) } | |||
|
|||
this.viewModel.outputs.showCreatedPasswordSnackback() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
snackbar* heh
fun newPassword(newPassword: String) | ||
|
||
/** Call when the user clicks the submit password button. */ | ||
fun submitPasswordClicked() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's rename this createPasswordClicked
for consistency.
import rx.subjects.BehaviorSubject | ||
import rx.subjects.PublishSubject | ||
|
||
const val CREATE_PASSWORD_MINIMUM_LENGTH = 6 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have this value now in 2 places. (The other is ChangePasswordViewModel
) Should we just have one constant that lives in StringUtils
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes I agree we should I'll add it to the StringUtils
Class.
@@ -54,6 +54,9 @@ interface LoginViewModel { | |||
/** Emits when a user has successfully changed their password and needs to login again. */ | |||
fun showChangedPasswordSnackbar(): Observable<Void> | |||
|
|||
/** Emits when a user has successfully created their psssword and needs to login again. */ | |||
fun showCreatedPasswordSnackback(): Observable<Void> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
snackbar :3
@@ -54,6 +54,9 @@ interface LoginViewModel { | |||
/** Emits when a user has successfully changed their password and needs to login again. */ | |||
fun showChangedPasswordSnackbar(): Observable<Void> | |||
|
|||
/** Emits when a user has successfully created their psssword and needs to login again. */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo in "password"
|
||
<TextView | ||
android:id="@+id/create_password_text_view" | ||
style="@style/TextInputPasswordLayout" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this is the right style.
# Conflicts: # app/src/main/AndroidManifest.xml # app/src/main/java/com/kickstarter/mock/services/MockApolloClient.kt # app/src/main/java/com/kickstarter/services/ApolloClientType.kt # app/src/main/java/com/kickstarter/services/KSApolloClient.kt # app/src/main/java/com/kickstarter/ui/activities/AccountActivity.kt
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great job!! Successfully tested on my account ٩( ᐛ )و
What ❓
AccountActivity
and you don't have a password you will see theCreate Account
row. If you do have a password you will see theChange Email
andChange Password
rows.Story 📖
Create Password
Create Password Event
See 👀
How to QA
Please refer to Trello description for testing instructions.