-
Notifications
You must be signed in to change notification settings - Fork 20
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
Dashboard issues of previous release - part 1 #1523
Comments
@1t5j0y I had suggested the DD-MM-YYYY format because we dont have translations for the months - and there would be users who will not know that as well(I have interacted with users who dont know English at all). didnt notice the as on date - as on date as well was before in DD-MM-YYYY format |
Pending the DD-MM-Year HH:MM formating, rest of the changes look fine as per code review stanadards. |
QA Reference video : |
For translations, refer this
Current screenshots:
Analysis notes:
Making hideTotalForProgram work in new MyDashboard might not be needed. Because from this and this looks like were done for 2 orgs - JSSCP and CK. But in JSSCP this setting is set to false and in CK, program and encounter type myDashboard filters itselves looks like are removed. Looks like so this setting is not relevant anymore.
The text was updated successfully, but these errors were encountered: