Skip to content

test(common): Engineering builds to test https://keyman-staging.com/keyboards search#15957

Open
darcywong00 wants to merge 1 commit into
masterfrom
test/keyman-staging
Open

test(common): Engineering builds to test https://keyman-staging.com/keyboards search#15957
darcywong00 wants to merge 1 commit into
masterfrom
test/keyman-staging

Conversation

@darcywong00
Copy link
Copy Markdown
Contributor

@darcywong00 darcywong00 commented May 14, 2026 •
edited
Loading

NOT TO REVIEW OR MERGE

For testing purposes of https://keyman-staging.com to verify

  • keyboard search is embedded view
  • keyboard search, download and install works
  • keyboard search by language works

Reverts most of #7227 (only changing keyman.com and keeping api and help sites)

User Testing

Install the PR build of Keyman for the platform

GROUP_ANDROID
GROUP_DEVELOPER
GROUP_IOS
GROUP_LINUX
GROUP_MACOS
GROUP_WINDOWS

(Note: for Developer, we want to verify the keyboard project clone from cloud functionality)

  • TEST_KEYBOARD_SEARCH - Verifies keyboard search functionality
  1. Launch Keyman
  2. Follow app menus to download keyboard
  3. Verify keyboard search is an embedded view of the keyboard search page
  4. Search for a keyboard and verify you can download and install it

Build-bot: release:all

@darcywong00
test(common): Engineering builds to test keyboard search
99db554 
For testing keyman-staging.com

Reverts most of #7227  (only changing keyman.com and  keeping api and help sites)
@darcywong00 darcywong00 added this to the A19S29 milestone May 14, 2026
@github-project-automation github-project-automation Bot moved this to Todo in Keyman May 14, 2026
@keymanapp-test-bot keymanapp-test-bot Bot added the user-test-missing User tests have not yet been defined for the PR label May 14, 2026
@keymanapp-test-bot
Copy link
Copy Markdown

keymanapp-test-bot Bot commented May 14, 2026 •
edited
Loading

User Test Results

Test specification and instructions

  • ✅ GROUP_ANDROID:

    1 tests PASSED

  • ✅ GROUP_DEVELOPER:

    1 tests PASSED

  • ⬜ GROUP_IOS:

    • ⬜ TEST_KEYBOARD_SEARCH (OPEN)

  • ⬜ GROUP_LINUX:

    • ⬜ TEST_KEYBOARD_SEARCH (OPEN)

  • ⬜ GROUP_MACOS:

    • ⬜ TEST_KEYBOARD_SEARCH (OPEN)

  • ✅ GROUP_WINDOWS:

    1 tests PASSED
Results Template 
# Test Results

### GROUP_IOS: 

* **TEST_KEYBOARD_SEARCH (OPEN):** notes

### GROUP_LINUX: 

* **TEST_KEYBOARD_SEARCH (OPEN):** notes

### GROUP_MACOS: 

* **TEST_KEYBOARD_SEARCH (OPEN):** notes

Test Artifacts

@keymanapp-test-bot keymanapp-test-bot Bot added has-user-test user-test-required User tests have not been completed and removed user-test-missing User tests have not yet been defined for the PR labels May 15, 2026
@darcywong00 darcywong00 marked this pull request as ready for review May 15, 2026 02:01
@Meng-Heng
Copy link
Copy Markdown
Contributor

Meng-Heng commented May 15, 2026 •
edited by darcywong00
Loading

Test Specs

  1. Android Studio Pixel 9 API 35
  2. Keyman for Android 19.0.232-alpha-test-15957

Test Results

GROUP_ANDROID

  • TEST_KEYBOARD_SEARCH (PASSED):

  1. Launch Keyman

  2. Follow app menus to download keyboard

  3. Verified: keyboard search is an embedded view of the keyboard search page

    Screenshot 2026-05-15 at 4 07 03 in the afternoon

  4. Search for a Khmer keyboard

  5. Verified: Able to download, install, and type Khmer Angkor

@keymanapp-test-bot keymanapp-test-bot Bot removed the user-test-required User tests have not been completed label May 15, 2026
@darcywong00
Copy link
Copy Markdown
Contributor Author

darcywong00 commented May 18, 2026 •
edited
Loading

PR build of Keyman for Windows notes

Configuration

  • Windows 11
  • keyman-19.0.232-alpha-test-15957.exe

Embedded search correctly showed and was able to search/install khmer_angkor.

However, the links on the root keyboard search opened in separate search window:

Opening DevTools on the embedded search window, I don't see the expected session cookie embed_keyboards_no_redirect_locale

image

@mcdurdin
Copy link
Copy Markdown
Member

mcdurdin commented May 18, 2026 •
edited
Loading

Opening DevTools on the embedded search window, I don't see the expected session cookie embed_keyboards_no_redirect_locale

That's because it's not a session cookie, it's a session variable. The PHPSESSID cookie tells PHP what the session id is, and then PHP looks it up in its internal session storage.

My bad. That is a cookie, not a PHP variable. But it's called embed_keyboards_no_locale_redirect! Looking deeper.

@mcdurdin mcdurdin mentioned this pull request May 18, 2026
Merged
@mcdurdin
Copy link
Copy Markdown
Member

mcdurdin commented May 18, 2026

Fix in keymanapp/keyman.com#743... another cookie prop needed.

@keymanapp-test-bot keymanapp-test-bot Bot added the user-test-required User tests have not been completed label May 18, 2026
@darcywong00
Copy link
Copy Markdown
Contributor Author

darcywong00 commented May 19, 2026

Test Specs

  • PR build of Keyman for Windows
  • Win 11 desktop

Test Results

GROUP_WINDOWS

  • TEST_KEYBOARD_SEARCH (PASSED):
  1. Launch Keyman
  2. Follow app menus to download keyboard
  3. Verify keyboard search is an embedded view of the keyboard search page
image
  1. Search for a keyboard and verify you can download and install it
  2. Verified - able to search and install khmer_angkor keyboard

The "Popular keyboards" and "All keyboards" links also remained in the embedded search

@darcywong00
Copy link
Copy Markdown
Contributor Author

darcywong00 commented May 19, 2026

Test Specs

  • PR build of Keyman Developer
  • Win 11 desktop

Test Results

GROUP_DEVELOPER

  • TEST_KEYBOARD_SEARCH (PASSED):
  1. Launch Developer
  2. Project --> New Project --> Clone project from Keyman Cloud
  3. Verify keyboard search is an embedded view of the keyboard search page
image
  1. Search for a keyboard and verify you can clone it
  2. Verified - able to search and clone khmer_angkor keyboard

Also verified "Popular keyboards" and "All keyboards" links remained in embedded search

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

Reviewers

No reviews

Assignees

No one assigned

Labels

android/engine/ android/ common/ DO-NOT-MERGE has-user-test ios/engine/ ios/ linux/config/ linux/ mac/ test Any acceptance test issue user-test-required User tests have not been completed

Projects

Keyman
Status: Todo

Milestone

A19S29

Development

Successfully merging this pull request may close these issues.

3 participants

@darcywong00 @Meng-Heng @mcdurdin