Medical Emergency Service Hotline 116 117 not linked as phone in Request TAN
See original GitHub issueAvoid duplicates
- Bug is not mentioned in the FAQ
- Bug is specific for Android only, for general issues / questions that apply to iOS and Android please raise them in the documentation repository
- Bug is not already reported in another issue
Technical details
- Device name: Samsung Galaxy A5 (2017) SM-A520F
- Android version: 8.0.0
- App version: 1.11.0 (also checked with branch release/1.12.x)
Describe the bug
In the Request TAN view the telephone number 116 117 is not linked as a phone number. For comparison, the same phone number is linked in the view shown via App Information > Technical hotline.
Steps to reproduce the issue
Note: autoLink used in CWA version 1.11.0 is dependent on the device’s locale setting. The following steps are with the device’s language set to English (United Kingdom). If the device’s language is set to English (United States) the telephone number 116 117 is not linked in the Technical Hotline view nor in the Request TAN view.
- Under “Have you been tested?” tap “NEXT STEPS”
- Scroll down to “No TAN yet?” and tap
- Scroll down to the paragraph “If you have any health-related questions, please contact your general practitioner or the medical emergency service hotline, telephone: 116 117.”
- Tap on the phone number 116 117 and note that nothing happens.
Compare to the link to phone number 116 117 shown on the Technical hotline view
Expected behaviour
After opening the view Request TAN through “Have you been tested?” > “NEXT STEPS” > “No TAN yet?” the phone number 116 117 should be underlined and tapping on it should bring up the Phone app.
The Request TAN view should show the phone number 116 117 the same way that the Technical Hotline view (accessed through three-dot overview menu > App Information > Technical hotline) shows it.
Possible Fix
Add the following attributes to TextView +id/submission_contact_body_other in layout\include_submission_contact.xml
so that the TextView is aligned with TextView @+id/information_contact_body_other in layout\fragment_information_contact.xml
android:autoLink="phone"
android:textColorLink="@color/colorTextTint"
I will submit a PR for this.
PR https://github.com/corona-warn-app/cwa-app-android/pull/2292
Additional context
This came to light due to examining phone number autoLinking under https://github.com/corona-warn-app/cwa-app-android/issues/2275
Internal tracking ID: EXPOSUREAPP-5007 This issue will be solved with EXPOSUREAPP-4992.
Issue Analytics
- State:
- Created 3 years ago
- Comments:6 (6 by maintainers)
Top GitHub Comments
@MikeMcC399 Thank you.
PR #2286 (EXPOSUREAPP-4992) is now in the 1.13.0 Milestone.