-
-
Notifications
You must be signed in to change notification settings - Fork 111
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
fix(developer): server download Keyman link #9822
fix(developer): server download Keyman link #9822
Conversation
User Test ResultsTest specification and instructions
Test Artifacts
|
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.
lgtm
@bharanidharanj I tried but was unable to reproduce this problem. Can you re-test this, and look to see if the Keyman Developer Server has started down in the system notification area? |
@keymanapp-test-bot retest |
Sure @mcdurdin . I will re-test it and will post my result. |
Changes in this pull request will be available for download in Keyman version 17.0.199-alpha |
Fixes #9702.
User Testing
Setup: you will need to have Keyman Developer Server running on your Windows computer for all these tests, in the same way as described in #9702. Then, you will need to locate the appropriate IP address on your local network for your computer, and ensure that the test devices are on the same local network. For example, on my machine, the address I use is http://192.168.129.109:8008:
Then, open that page on the target device and test the Download Keyman link there.
GROUP_WINDOWS: Test on Windows
GROUP_MAC: Test on MacOS
GROUP_ANDROID: Test on MacOS
GROUP_IOS: Test on iOS
GROUP_LINUX: Test on Linux
TEST_LINK: Test ths the Download Keyman link starts a download for Keyman or loads a page appropriate for the device