Skip to content
This repository has been archived by the owner on Nov 8, 2018. It is now read-only.

Fixed issue_276 #375

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Conversation

riddhik84
Copy link

  • Contries names on login screen
  • SSM,SARL,PCMO numbers according to contry on get help now screen
  • Disable button when phone number is not available
  • Refresh get help screen now when country is changed from settings

- Contries names on login screen
- SSM,SARL,PCMO numbers according to contry on get help now screen
- Disable button when phone number is not available
- Refresh get help screen now when country is changed from settings
@riddhik84
Copy link
Author

Any update on this fix?


return rootView;
}


public void readPhoneNumbersCSV(String filename) {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

File read should be run as a background task

<item>Timor-Leste</item>
<item>Togo</item>
<item>Tonga</item>
<item>Uganda</item>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this repeated?

<item>Timor-Leste</item>
<item>Togo</item>
<item>Tonga</item>
<item>Uganda</item>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

here as well?

@@ -1,15 +1,147 @@
<?xml version="1.0" encoding="utf-8"?>
<resources>
<string-array name="countryArray">

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there specific reason for having two arrays as country names and country array? Doesn't both these arrays reflects the same content??

@sandarumk
Copy link

@riddhik84 Any updates?

@sandarumk
Copy link

If there are no updates on this one, I am going to close. Please raise any concerns.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants