This repository has been archived by the owner on Dec 14, 2021. It is now read-only.
Add null text value for empty usernames #902
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jhugman
commented
Sep 5, 2019
| @@ -33,7 +33,7 @@ buildscript { | |||
| } | |||
| } | |||
| dependencies { | |||
| classpath 'com.android.tools.build:gradle:3.4.2' | |||
| classpath 'com.android.tools.build:gradle:3.5.0' | |||
There was a problem hiding this comment.
Not necessary for this PR, but upgrading as part of our normal code cycle.
jhugman
force-pushed
the
jhugman/758-autofill-empty-username
branch
from
1a22c5a to
6601258
Compare
|
Tested for crash with STR in #488 (fixed by #496)— no crash. Please review when ready, @eliserichards |
eliserichards
approved these changes
Sep 9, 2019
| return node.htmlInfo?.tag?.toLowerCase() == "input" | ||
| } | ||
| override fun isHtmlInputField(node: ViewNode) = | ||
| // Use English locale, as the HTML tags are all in English. |
jhugman
force-pushed
the
jhugman/758-autofill-empty-username
branch
from
6601258 to
718644c
Compare
eliserichards
pushed a commit
that referenced
this pull request
Sep 11, 2019
* Add release notes for 1.1.2.4405 * Bump version to 1.2.0 * Add ignore to the flaky test. This needs to be fixed later. (#793) * 626, 627: Add kebab menu for delete (#764) * Add kebab menu icon and strings. Set up fragment for addition to toolbar. * Create fragment and dialog. * Add confirmation dialog, actions, and presenters * Routing * Ensure item is actually deleted and item list is refreshed before navigating back. * Refactor and remove edit for now * Keeping edit button, just not doing any functionality with it * Test for item detail presenter * Ellipsize long item titles in detail view * Delete confirmation toast * Delete toast and tests * Consumable toast notification for delete * Specify return type on Optional extension * ItemListPresenterTest needed a consumable for the delete toast test * Cleanup * Fix toast notifications * Import strings from android-l10n (#809) State: mozilla-l10n/android-l10n@63a5240 * 806-uitest-add-delete (#808) * Adjust spinner dropdown size to fit longer words (l10n) (#812) * Ellipsize text in item list (#811) * 810: Reset support when syncing (#826) * Reset support when info is null * Reset support every time we sync. * Removing timeout to be fixed in #791 (#833) * Removing timeout to be fixed in #791 * Remove strings * 828-l10nScreenshotsTest-delete (#829) * Add dot to sentence (#837) * 814: Telemetry for sync (#835) * Add telemetry actions for sync, list update, and error states. * Sync timeout instead of sync end * Add sync complete event * Add errors and ids to telemetry calls * Update metrics docs to reflect telemetry sync changes (#841) * Revert delete PRs (#836) * Revert "828-l10nScreenshotsTest-delete (#829)" This reverts commit bab9fe8. * Revert "806-uitest-add-delete (#808)" This reverts commit ec56009. * Revert "626, 627: Add kebab menu for delete (#764)" This reverts commit 17605c3. * Remove unused imports and fix lint error. * Rebase and lint * Update to version 1.1.3 (#844) * Update version to 1.1.3 * Add release notes for 1.1.3 * Update releases.md (#847) * Add more context to release notes * Add descriptions for proguard rules upload. * Revert "Revert delete PRs (#836)" (#842) This reverts commit 974a585. * 816: string updates for localization (#852) * Revert "Revert delete PRs (#836)" This reverts commit 974a585. * Lockwise placeholder in delete_description and descriptive comment in edit string * Update app services, android components, and megazord configuration (#865) * Update app services, android components, and megazord configuration * Update a-c to v7.0.0 * Send hashed UID ping when sync completes successfully. * Remove handling for synctelemetryping, to be completed in separate issue. Update AS to 0.37.1 * Update Readme with l10n process (#872) This is something we've been adding to all Readmes of mobile repos to direct people to Pontoon for localization (vs GitHub repos, where work isn't supposed to happen). * 867 - Fix obvious NPE sync crash (#884) * 871: Create infrastructure for feature flags (#882) * Delete feature flag * Move feature flags to separate support file * Create support to check build configs. Create flags for CUD * Add telemetry probes to syncIfRequired trigger syncs (#887) * Import strings from l10n (#893) * Import strings from android-l10n State: mozilla-l10n/android-l10n@349efdb * Commenting unused string for Espanol * 791: Remove fake timeout from sync (#889) * Remove fake timeout from sync * Remove all sync timeout strings * Rebase and lint * Fail fast if the edge between to routes is not in the nav graph. (#895) * Auto-signin with mozilla AuthProvider (#879) — behind a feature flag. * WIP Automatic FxA sign-in support * Cleanup welcome fragment/account detection * Upgrade to Android Components 9.0.0 * Ensure landable, even if not releasable * gradlew lint * 628: edit view and routing (#843) * Edit xml layout * Add route, presenter, and fragment Button clicks and dialog Update dependencies Edit presenter tests Routing stuff Usable state with spinner Buttons working Password visibility in edit mode Lint View Create popup item and insert menu into detail view's kebab buttton * Popup menu and click listener Dropdown menu and formatting Update list - in progress DataStoreTest - mocked up test, stuck on init Datastore update item detail test * Save edited changes Lint Update dependencies Edit unit tests Add sync to datastore editing. Update var name in robot and screenshot tests Reformatting dropdown menu and edit view Adding inclusive popup to edit->detail view nav definition Address nullable server passwords for delete and edit. Remove unused string. Remove learn more clicks from test Refactoring pushError into a helper method. * Add fxalogin to autofill onboarding route. (#901) * 798: routing back to itemlist on login/search/feedback forms (#890) * Investigation * Add locking routes. * Explicitly ignore autofill on search and edit fields * Ignore autofill on webviews * Remove comments and clean up lint * Add null text value for empty usernames (#902) * Add null text value for empty usernames * gradlew lint * 907 fix ui tests (#908) * 907: fix testSettings * fixing routing issues * fixing more routes * changing button name and adding route * fixing conflicts after rebase * Update release-notes.md with v1.1.4 * Fix lint errors * Update release-notes.md to correct bitrise version
eliserichards
pushed a commit
that referenced
this pull request
Sep 12, 2019
* Add release notes for 1.1.2.4405 * Bump version to 1.2.0 * Add ignore to the flaky test. This needs to be fixed later. (#793) * 626, 627: Add kebab menu for delete (#764) * Add kebab menu icon and strings. Set up fragment for addition to toolbar. * Create fragment and dialog. * Add confirmation dialog, actions, and presenters * Routing * Ensure item is actually deleted and item list is refreshed before navigating back. * Refactor and remove edit for now * Keeping edit button, just not doing any functionality with it * Test for item detail presenter * Ellipsize long item titles in detail view * Delete confirmation toast * Delete toast and tests * Consumable toast notification for delete * Specify return type on Optional extension * ItemListPresenterTest needed a consumable for the delete toast test * Cleanup * Fix toast notifications * Import strings from android-l10n (#809) State: mozilla-l10n/android-l10n@63a5240 * 806-uitest-add-delete (#808) * Adjust spinner dropdown size to fit longer words (l10n) (#812) * Ellipsize text in item list (#811) * 810: Reset support when syncing (#826) * Reset support when info is null * Reset support every time we sync. * Removing timeout to be fixed in #791 (#833) * Removing timeout to be fixed in #791 * Remove strings * 828-l10nScreenshotsTest-delete (#829) * Add dot to sentence (#837) * 814: Telemetry for sync (#835) * Add telemetry actions for sync, list update, and error states. * Sync timeout instead of sync end * Add sync complete event * Add errors and ids to telemetry calls * Update metrics docs to reflect telemetry sync changes (#841) * Revert delete PRs (#836) * Revert "828-l10nScreenshotsTest-delete (#829)" This reverts commit bab9fe8. * Revert "806-uitest-add-delete (#808)" This reverts commit ec56009. * Revert "626, 627: Add kebab menu for delete (#764)" This reverts commit 17605c3. * Remove unused imports and fix lint error. * Rebase and lint * Update to version 1.1.3 (#844) * Update version to 1.1.3 * Add release notes for 1.1.3 * Update releases.md (#847) * Add more context to release notes * Add descriptions for proguard rules upload. * Revert "Revert delete PRs (#836)" (#842) This reverts commit 974a585. * 816: string updates for localization (#852) * Revert "Revert delete PRs (#836)" This reverts commit 974a585. * Lockwise placeholder in delete_description and descriptive comment in edit string * Update app services, android components, and megazord configuration (#865) * Update app services, android components, and megazord configuration * Update a-c to v7.0.0 * Send hashed UID ping when sync completes successfully. * Remove handling for synctelemetryping, to be completed in separate issue. Update AS to 0.37.1 * Update Readme with l10n process (#872) This is something we've been adding to all Readmes of mobile repos to direct people to Pontoon for localization (vs GitHub repos, where work isn't supposed to happen). * 867 - Fix obvious NPE sync crash (#884) * 871: Create infrastructure for feature flags (#882) * Delete feature flag * Move feature flags to separate support file * Create support to check build configs. Create flags for CUD * Add telemetry probes to syncIfRequired trigger syncs (#887) * Import strings from l10n (#893) * Import strings from android-l10n State: mozilla-l10n/android-l10n@349efdb * Commenting unused string for Espanol * 791: Remove fake timeout from sync (#889) * Remove fake timeout from sync * Remove all sync timeout strings * Rebase and lint * Fail fast if the edge between to routes is not in the nav graph. (#895) * Auto-signin with mozilla AuthProvider (#879) — behind a feature flag. * WIP Automatic FxA sign-in support * Cleanup welcome fragment/account detection * Upgrade to Android Components 9.0.0 * Ensure landable, even if not releasable * gradlew lint * 628: edit view and routing (#843) * Edit xml layout * Add route, presenter, and fragment Button clicks and dialog Update dependencies Edit presenter tests Routing stuff Usable state with spinner Buttons working Password visibility in edit mode Lint View Create popup item and insert menu into detail view's kebab buttton * Popup menu and click listener Dropdown menu and formatting Update list - in progress DataStoreTest - mocked up test, stuck on init Datastore update item detail test * Save edited changes Lint Update dependencies Edit unit tests Add sync to datastore editing. Update var name in robot and screenshot tests Reformatting dropdown menu and edit view Adding inclusive popup to edit->detail view nav definition Address nullable server passwords for delete and edit. Remove unused string. Remove learn more clicks from test Refactoring pushError into a helper method. * Add fxalogin to autofill onboarding route. (#901) * 798: routing back to itemlist on login/search/feedback forms (#890) * Investigation * Add locking routes. * Explicitly ignore autofill on search and edit fields * Ignore autofill on webviews * Remove comments and clean up lint * Add null text value for empty usernames (#902) * Add null text value for empty usernames * gradlew lint * 907 fix ui tests (#908) * 907: fix testSettings * fixing routing issues * fixing more routes * changing button name and adding route * fixing conflicts after rebase * Update release-notes.md with v1.1.4 * Update release notes for v2.0.0 * Update version code and version name to v2.0.0
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #758.