Skip to content

Fixes #8002: Add ability to change username #10497

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.

Sign up for GitHub

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

Merged
merged 35 commits into from
Sep 9, 2019
Merged

Fixes #8002: Add ability to change username #10497

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
35 commits
Select commit Hold shift + click to select a range
8817437
Added dismissAfterConfirm flag.
jd-alexander Aug 20, 2019
ef150d1
Merge pull request #10414 from jd-alexander/fullscreen_dialog_confirm…
maxme Aug 21, 2019
3362f72
Made UsernameChangerFullScreenDialogFragment abstract so the function…
jd-alexander Aug 19, 2019
482d107
Updated UsernameChangerFullScreenDialogFragment to utilize the Androi…
jd-alexander Aug 19, 2019
e9c1cdb
Created test for UsernameChangerFullScreenDialogFragment
jd-alexander Aug 19, 2019
4f52d3a
Performed code cleanup and improvements.
jd-alexander Aug 19, 2019
e2544f4
Fixed ktlint error.
jd-alexander Aug 19, 2019
744c00b
Merge pull request #10409 from jd-alexander/8002_make_suggestions_fra…
maxme Aug 22, 2019
7aa29d7
Merge branch 'develop' into issue/8002-change-username
malinajirka Aug 26, 2019
ca1e8a1
Implemented change username functionality in the Account Settings Scr…
jd-alexander Aug 26, 2019
5455d51
dismiss progress on username success and set username to preference c…
jd-alexander Aug 26, 2019
9274521
Used R.string instead of string only due to lint errors. Fetch accoun…
jd-alexander Aug 26, 2019
93e165e
Save button is disabled until a new username is selected.
jd-alexander Aug 27, 2019
c354167
Added cancellation dialog that's shown when user cancels progress dia…
jd-alexander Aug 27, 2019
d586c5a
added analytics events to all the api calls.
jd-alexander Aug 27, 2019
01c8d81
Resolved syntax suggestions, typos and made other improvements to str…
jd-alexander Aug 28, 2019
deed7c3
Made fields and several methods in base class private. Created the ne…
jd-alexander Aug 29, 2019
4a1a261
Added event name for change username stats
jd-alexander Aug 29, 2019
4e33a33
Disabled the username changer test.
jd-alexander Aug 29, 2019
4b2e8b2
Made onConfirmClicked abstract in base class since it's clear both cl…
jd-alexander Aug 29, 2019
8d7fb5d
Fixed lint error where comment line was too long.
jd-alexander Aug 29, 2019
15e56ff
Utilize the updated username for bundles and account refreshes.
jd-alexander Aug 29, 2019
4c23ba4
simplified bundle creation with kotlin's apply scoping function.
jd-alexander Aug 29, 2019
1332f35
Revert "Utilize the updated username for bundles and account refreshes."
jd-alexander Aug 29, 2019
f450a40
Simplified onConfirmClicked with an abstract method for the shared lo…
jd-alexander Aug 29, 2019
62c2f08
Fixed progress dialog issue and made code style fix.
jd-alexander Aug 29, 2019
acf156e
Replaced Toast with SnackBar for username changed event.
jd-alexander Aug 29, 2019
97fe60b
Revert "Added dismissAfterConfirm flag."
jd-alexander Aug 20, 2019
a3f5c51
Removed Username Changer test and the associated modifications that s…
jd-alexander Aug 30, 2019
a1b1906
Merge pull request #10436 from jd-alexander/8002_settings_username_ch…
malinajirka Aug 30, 2019
8827832
Update release notes - change username
malinajirka Aug 30, 2019
02a1ba0
Add Other post type to posts and pages
planarvoid Sep 4, 2019
15d6122
Merge branch 'develop' into issue/8002-change-username
maxme Sep 9, 2019
a799078
Merge branch 'feature/add_other_post_type_to_posts_and_pages' of http…
maxme Sep 9, 2019
b008b01
Update FluxC version to 1.4.0-beta-1
maxme Sep 9, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
added analytics events to all the api calls.
  • Loading branch information
@jd-alexander
jd-alexander committed Aug 27, 2019
commit d586c5a7fb1dbaa9900ca951e432438bf18187fb
12 changes: 6 additions & 6 deletions ...src/main/java/org/wordpress/android/ui/accounts/signup/SettingsUsernameChangerFragment.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,9 +17,9 @@ import org.greenrobot.eventbus.ThreadMode
import org.wordpress.android.R
import org.wordpress.android.R.style
import org.wordpress.android.analytics.AnalyticsTracker
import org.wordpress.android.analytics.AnalyticsTracker.Stat
import org.wordpress.android.analytics.AnalyticsTracker.Stat.SIGNUP_EMAIL_EPILOGUE_UPDATE_USERNAME_FAILED
import org.wordpress.android.analytics.AnalyticsTracker.Stat.SIGNUP_SOCIAL_EPILOGUE_USERNAME_SUGGESTIONS_FAILED
import org.wordpress.android.analytics.AnalyticsTracker.Stat.ACCOUNT_SETTINGS_CHANGE_USERNAME_FAILED
import org.wordpress.android.analytics.AnalyticsTracker.Stat.ACCOUNT_SETTINGS_CHANGE_USERNAME_SUCCEEDED
import org.wordpress.android.analytics.AnalyticsTracker.Stat.ACCOUNT_SETTINGS_CHANGE_USERNAME_SUGGESTIONS_FAILED
import org.wordpress.android.fluxc.generated.AccountActionBuilder
import org.wordpress.android.fluxc.store.AccountStore.AccountUsernameActionType.KEEP_OLD_SITE_AND_ADDRESS
import org.wordpress.android.fluxc.store.AccountStore.OnUsernameChanged
Expand All @@ -38,7 +38,7 @@ class SettingsUsernameChangerFragment : BaseUsernameChangerFullScreenDialogFragm
private lateinit var dialogController: FullScreenDialogController
private var progressDialog: ProgressDialog? = null

override fun getSuggestionsFailedStat() = SIGNUP_SOCIAL_EPILOGUE_USERNAME_SUGGESTIONS_FAILED
override fun getSuggestionsFailedStat() = ACCOUNT_SETTINGS_CHANGE_USERNAME_SUGGESTIONS_FAILED
override fun canHeaderTextLiveUpdate() = false
override fun getHeaderText(username: String?, display: String?): Spanned = HtmlCompat.fromHtml(
String.format(
Expand Down Expand Up @@ -140,18 +140,18 @@ class SettingsUsernameChangerFragment : BaseUsernameChangerFullScreenDialogFragm
@Subscribe(threadMode = ThreadMode.MAIN)
fun onUsernameChanged(event: OnUsernameChanged) {
if (event.isError) {
AnalyticsTracker.track(SIGNUP_EMAIL_EPILOGUE_UPDATE_USERNAME_FAILED)
AnalyticsTracker.track(ACCOUNT_SETTINGS_CHANGE_USERNAME_FAILED)
AppLog.e(
T.API, "SettingsUsernameChangerFragment.onUsernameChanged: " +
event.error.type + " - " + event.error.message
)
endProgress()
showErrorDialog(SpannableStringBuilder(getString(R.string.signup_epilogue_error_generic)))
} else if (event.username != null) {
AnalyticsTracker.track(ACCOUNT_SETTINGS_CHANGE_USERNAME_SUCCEEDED)
endProgress()
val result = Bundle().apply { putString(RESULT_USERNAME, event.username) }
dialogController.confirm(result)
AnalyticsTracker.track(Stat.SIGNUP_EMAIL_EPILOGUE_UPDATE_USERNAME_SUCCEEDED)
}
}

Expand Down
3 changes: 3 additions & 0 deletions ...cs/WordPressAnalytics/src/main/java/org/wordpress/android/analytics/AnalyticsTracker.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -273,6 +273,9 @@ public enum Stat {
OPENED_MEDIA_LIBRARY,
OPENED_BLOG_SETTINGS,
OPENED_ACCOUNT_SETTINGS,
ACCOUNT_SETTINGS_CHANGE_USERNAME_SUCCEEDED,
ACCOUNT_SETTINGS_CHANGE_USERNAME_FAILED,
ACCOUNT_SETTINGS_CHANGE_USERNAME_SUGGESTIONS_FAILED,
OPENED_APP_SETTINGS,
OPENED_MY_PROFILE,
OPENED_PEOPLE_MANAGEMENT,
Expand Down