Skip to content

chore(android/app): Revert #8291#8314

Merged
mcdurdin merged 1 commit intomasterfrom
chore/android/revert-8291
Feb 25, 2023
Merged

chore(android/app): Revert #8291#8314
mcdurdin merged 1 commit intomasterfrom
chore/android/revert-8291

Conversation

@darcywong00
Copy link
Copy Markdown
Contributor

Reverts #8291 and fixes #8306
I will have to re-open #8229 for investigation...

This reverts commit dc7603a.

User Testing

Setup - install the PR build of Keyman for Android

  • TEST_INSTALL - Verifies reported issue does not appear
  1. Install the PR build of Keyman for Android
  2. Open Keyman In-App.
  3. Open Settings menu.
  4. Enable Keyman as system-wide keyboard.
  5. Enable 'Set Keyman as default keyboard'.
    6 Add 'Khmer Angkor' Keyboard using 'Install Keyboard or Dictionary' option.
  6. Verify blank screen does not appear

@keymanapp-test-bot keymanapp-test-bot bot added has-user-test user-test-required User tests have not been completed labels Feb 24, 2023
@keymanapp-test-bot
Copy link
Copy Markdown

keymanapp-test-bot bot commented Feb 24, 2023

User Test Results

Test specification and instructions

  • TEST_INSTALL (OPEN)
Results Template
# Test Results

* **TEST_INSTALL (OPEN):** notes

Test Artifacts

@mcdurdin
Copy link
Copy Markdown
Member

Let's go ahead and merge this so that we can unbreak Android on master. Given it just reverts the previous thing which broke master, it should be good to merge.

@mcdurdin mcdurdin merged commit 8c5bd34 into master Feb 25, 2023
@mcdurdin mcdurdin deleted the chore/android/revert-8291 branch February 25, 2023 23:17
@keyman-server
Copy link
Copy Markdown
Collaborator

Changes in this pull request will be available for download in Keyman version 17.0.57-alpha

@bharanidharanj
Copy link
Copy Markdown

bharanidharanj commented Feb 27, 2023

Test Results

  • TEST_INSTALL (FAILED): Tested with the attached PR build (17.0.57-alpha) in Android 12 / API 31 emulator and here is my observation: 1. Getting the same issue (ie., Blank keyboard appears after Khmer angkor Keyboard installation) after installing a keyboard. 2. I noticed that some error message appears 'unavailable resource, 'could not connect server' during the installation. Seems to be an issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug(android): Keyboard installation problem occurs after enabling and setting keyman as the default keyboard

4 participants