Wrong order for synopsis

[kuvakina]

The order we see on Patient Summary page and on the issue page is different.

For OneLondon :

For Helm-RA :

[kuvakina]

@tony-shannon @PhilBarrett
on this task Bogdan approx need 2-3h

[tony-shannon]

Bogdan to investigate this one please

[BogdanScherban]

@tony-shannon @DmitrySolyannik

I solve this issue in Showcase version.

I changed sort order in synopsis (to opposite):

And added default sort by dateCreated (opposite):

But I can't do this for plugins: Problems, Contacts, Mdt - because dateCreated fiels is absent in these plugins.

So, I suggest:

1. Add dateCreated for all plugins, where it is absent;
2. Save this changes to Showcase now. Later we will copy all common changes from Showcase to master-branch and merge it to NES, London and Helm.

[tony-shannon]

ok thanks, need estimate of effort for those suggestions please

[BogdanScherban]

@tony-shannon

#1 - it needs @DmitrySolyannik estimation, because it is server-side issue.

#2 - I need in 2-3 h for this work:

1. Create new branch based on Master
2. Copy common changes from Showcase to Master
3. Check, that everything works correctly
4. Merge new branch to NES, London and Helm
5. Check, that NES, London and Helm work correctly after merging and update them.
6. Create PR

[BogdanScherban]

@tony-shannon @PhilBarrett

I added required sorting at the client side (by dateCreated in opposite order).

But in some cases orders in synopsis and in table list aren't correspond.
For example, Medications

:

You can see, that positions 1-3 are similar, but position 4 is different - 'Testing' in synopsis and 'Amoxicillin 500mg capsules' in table list.

The reason - 'Testing' and 'Amoxicillin 500mg capsules' have equal dateCreated - '08-06-2016'.

For this reason I suggest @DmitrySolyannik to add double sorting at the server side - by dateCreated (first) and by name (second). For both cases - for synopsis and for table list.

[tony-shannon]

@DmitrySolyannik could you please discuss this with @robtweed to get a view from both of you on whether this is a client or server side issue please

[kuvakina]

@tony-shannon Dima added his changes, @BogdanScherban could you please add changes on your side?

[kuvakina]

@tony-shannon @PhilBarrett

As I understand it's working correct now, but the problem if that we have a lot issues in one heading with the same date. If we create issues for new patient and all issues won't be "today", we won't see the problem at all

For now, I'll move it to Review until we test it with new user

[PhilBarrett]

@kuvakina can you confirm if this is ok to close now please?