Skip to content
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

S2S_Revision of the 'share requests' directory_ Part 1 [3] #9755

Closed
8 tasks
Tracked by #10739
SahaLinaPrueger opened this issue Jul 6, 2022 · 24 comments · Fixed by #10161 or #10332
Closed
8 tasks
Tracked by #10739

S2S_Revision of the 'share requests' directory_ Part 1 [3] #9755

SahaLinaPrueger opened this issue Jul 6, 2022 · 24 comments · Fixed by #10161 or #10332
Assignees
Labels
backend Affects the web backend change A change of an existing feature (ticket type) qa-verified Issue has been tested and verified by QA s2s Issues related to the SORMAS2SORMAS functionality vaadin-app Affects the Vaadin application

Comments

@SahaLinaPrueger
Copy link

SahaLinaPrueger commented Jul 6, 2022

Problem Description

The 'share request' directory has no filter options. That is a problem, because after a while the directory becomes confusing.

At the moment the 'share requests' directory only shows incoming share requests and not the outgoing ones. That is a problem, because the health departments have no option to get an overview about shared cases/contacts and cannot track the shared cases/contacts.

Also the layout of the overview table does not include necessary information's, that the health departments need to be able to work.

Proposed Change

  • 1. Change the name from 'share requests' to 'share directory' (German: Übergabeverzeichnis)
  • 2. Add the following filter options (layout and location are the same as in the case directory)
Filter Typ Impact
Status Dropdown All - No records are filtered.
Pending - Only shares with the value "Pending" in "Status" are displayed.
Accepted - Only shares with the value "Accepted" in "Status" are displayed.
Rejected - Only those shares are displayed that have the value "Rejected" in "Status".
Error - Only shares with the value "Error" in "Status" are displayed.

"" is the default setting, which shows all entries
  • 3. Delete the buttons 'Pending' and 'Accepted'
    image

  • 4. Add the buttons 'Incoming' (German: Eingehend) in the place where the button 'Pending' was previously seen.
    image

Button Incoming
Impact Only the incoming shares are displayed
Comment -This filter is active by default when the share directory is opened for the first time in the session.
-The total number of results for this filter are displayed on the button if the button has been clicked/ is active.
-Clicking on the button 'Incoming' should show all cases/contacts that where shared from another health department to the own health department (status can be: accepted and pending- if user clicks 'reject' the share has to disappear because of data protection reasons).
  • 5. Add the button 'Outgoing' (German: Ausgehend) in the place where the button 'Accepted' was previously seen.
    image
Button Outgoing
Impact Only the outgoing shares are displayed
Comment -The total number of results for this filter are displayed on the button if the button has been clicked/ is active
-Clicking on the button 'Outgoing' should show all cases/ contacts that where shared with another health department (status can be: accepted, pending, revoked and rejected)
  • 6. Change the configuration of the button 'All': This button should include all 'Incoming' cases/contacts and all 'Outgoing' cases/contacts.
    image
Button All
Impact Both incoming and outgoing shares are displayed
Comment The total number of results for this filter are displayed on the button if the button is clicked
  • 7. The layout of the overview table should include the following columns:
Share ID (German: Übergabe ID) Share Date Typ From / To
(Counterpart)
Sender Name Ownership handed over Status Direction Comment
eye button
A double click or a single click on the "eye" button opens the preview of a share (is already implemented)
First 6 sign of Share ID (already implemented) The date of the share, localised according to current language (already implemented) 'Case' or 'Contact' is shown (already implemented) Name of other counterpart for example 'LK Würzburg' if 'LK Würzburg' is the sender organisation or received organisation Name of Person who has initiated the share (Format: Last name, First name) 'Yes' or 'No' is shown (already implemented) Current status of the share 'accepted' 'pending' or 'rejected'. Is updated depending on what the target system decides (if the share is outgoing) or what the source system decides (if the share is incoming). 'Incoming' or 'Outgoing'. Accept and Reject button as they are already implemented (1.73.1) Text field as already implemented (1.73.1)
  • 7.1 All columns can be sorted in ascending and descending order.

Acceptance Criteria

Implementation Details

Additional Information

@SahaLinaPrueger SahaLinaPrueger added change A change of an existing feature (ticket type) s2s Issues related to the SORMAS2SORMAS functionality labels Jul 6, 2022
@SahaLinaPrueger SahaLinaPrueger changed the title S2S_ First Revision of the 'share requests' directory [in progress] S2S_Revision of the 'share requests' directory_ Part 1 Jul 7, 2022
@SahaLinaPrueger
Copy link
Author

@JaquM-HZI

@markusmann-vg markusmann-vg changed the title S2S_Revision of the 'share requests' directory_ Part 1 S2S_Revision of the 'share requests' directory_ Part 1 [3] Jul 27, 2022
@AndyBakcsy-she AndyBakcsy-she added this to Backlog in SORMAS Team 4 - TEC - Iteration Backlog via automation Jul 27, 2022
@leventegal-she leventegal-she self-assigned this Aug 10, 2022
@leventegal-she leventegal-she moved this from Backlog to In Progress in SORMAS Team 4 - TEC - Iteration Backlog Aug 10, 2022
@leventegal-she
Copy link
Contributor

leventegal-she commented Aug 10, 2022

@SahaLinaPrueger It is not possible to have both incoming and outgoing requests at the same time because incoming and outgoing requests are stored in different database tables and those can't be merged.

Is it ok if we remove the 'All' option and the direction column from the table?

@SahaLinaPrueger
Copy link
Author

That would be a pity for the workflow and for the consistency of the sormas UI.
Do we have another option?

If we do not find another solution:
We can remove the direction column from the table if the buttons "incoming" and "outgoing" are fixed when scrolling down. And maybe we have to implement the column one day, if the column is necessary for showing this column in an excel file, if we implement an export one day. @Jan-Boehme what do you think?

Another point (to clarify): The column "From / To (Counterpart)" shows the server instance name not the jurisdiction, because we decided yesterday to share with server instance not with jurisdiction.

@leventegal-she
Copy link
Contributor

To make "incoming" and "outgoing" buttons fixed I can:

  1. Similar to case view modes: S2S_1
  2. With tabs like: S2S_2

Which one would you prefer?

@SahaLinaPrueger @MateStrysewske @MartinWahnschaffe @markusmann-vg

@SORMAS-JanBoehme
Copy link

@leventegal-she Please implement option Nr.1 so it is consistent across all directories.
The column "Direction" can be removed if we do not have an "All" view.

Just for me to better understand the problem here.
Why exactly are we not able to show information in one table in the UI that is stored in different database tables?
Is this a Vaadin thing?

@SahaLinaPrueger
Copy link
Author

SahaLinaPrueger commented Aug 11, 2022

If we implement 1 or 2 the filter option "status" doubles itself with the buttons "pending" and "accepted".

@SORMAS-JanBoehme
Copy link

That is correct because we have the filter for "Status" we do not need the buttons "Pending" and "Accepted" anymore. They can be removed.

@SahaLinaPrueger
Copy link
Author

SahaLinaPrueger commented Aug 11, 2022

@leventegal-she
So please implement Nr. 1 and remove the buttons "all", "pending" and "accepted".

The filter status stays, so please implement

  • 2. Add the following filter options (layout and location are the same as in the case directory)
Filter Typ Impact
Status Dropdown All - No records are filtered.
Pending - Only shares with the value "Pending" in "Status" are displayed.
Accepted - Only shares with the value "Accepted" in "Status" are displayed.
Rejected - Only those shares are displayed that have the value "Rejected" in "Status".
"" is the default setting, which shows all entries

@SahaLinaPrueger
Copy link
Author

And thank you very much for developing Nr. 1 and Nr. 2 !

@leventegal-she
Copy link
Contributor

@Jan-Boehme

Just for me to better understand the problem here.
Why exactly are we not able to show information in one table in the UI that is stored in different database tables?
Is this a Vaadin thing?

The data structure for incoming and outgoing requests are different and is not possible on the backend to query more than one database table at once and have lazy loading with sort.
As there will be a lot of requests it is not performant if we load both tables fully and manage sorting and and pagination in the memory.

@JonasCir
Copy link
Contributor

@AndyBakcsy-she Can you add the correct milestone here? I'm not certain if this should go into 1.75 or 1.76?

@dragosb-vg
Copy link

dragosb-vg commented Sep 19, 2022

Validated on test.de1 & test.de2 on Version: 1.76.0-SNAPSHOT (378ad16).
Summary of elements implemented/changed:

  1. Title 'Share requests' changed to 'Share Directory' (EN+DE)
  2. 'Status' filter dropdown contains:
  • for Incoming: - / Pending / Accepted
  • for Outgoing: - / Pending / Accepted / Rejected / Revoked
  1. Buttons Pending' and 'Accepted' were removed
  2. -5. 'Incoming'/'Outgoing' filters implemented as buttons (Incoming = Default)
  • Buttons text {'Eingehend'/'Ausgehend') not yet translated for users having DE language
  1. 'All' button removed
  2. Table Layout fields:
  • ShareID/CreationDate/Type/From-To(ServerName)/SenderName/HandOver/Status/(for revoke button)/Comment
  • Direction column removed

@StefanKock StefanKock added backend Affects the web backend vaadin-app Affects the Vaadin application labels Sep 25, 2022
@SahaLinaPrueger
Copy link
Author

Buttons text {'Eingehend'/'Ausgehend') not yet translated for users having DE language

Translations are not yet available on crowdIn. Please provide.

@SahaLinaPrueger
Copy link
Author

Line does not have the same height everywhere and layout changes greatly depending on filter setting. If it is possible, it would be great if the layout remains the same. Concerns incoming and outgoing.

revision share directory

SORMAS Team 4 - TEC - Iteration Backlog automation moved this from Done to In Progress Sep 28, 2022
@SahaLinaPrueger
Copy link
Author

@leventegal-she And another thing: Why is the column 'creation date' not renamed to 'share date' as specified? We found 'creation date' confusing because it was not clear what the word 'creation' referred to, so we thought 'share date' would be better. Did you also find 'share date' misleading and 'delivery date' or something similar would be better? Or was it just forgotten to change?

@markusmann-vg
Copy link

markusmann-vg commented Oct 5, 2022

@leventegal-she: open task for this ticket, before it can be closed. Additional requirements will be taken care of in a new ticket

  • Rename column 'creation date' to 'share date'

@MartinWahnschaffe
Copy link
Contributor

@leventegal-she @markusmann-vg I guess changing the caption of the creationDate field is sufficient?

@leventegal-she
Copy link
Contributor

@leventegal-she @markusmann-vg I guess changing the caption of the creationDate field is sufficient?

OK.

For the alignment issue the only solution I see is to set fixed size on some of the columns so there will be less changes when filtering the grid

leventegal-she pushed a commit that referenced this issue Oct 5, 2022
SORMAS Team 4 - TEC - Iteration Backlog automation moved this from Backlog to Testing Oct 5, 2022
@AndyBakcsy-she
Copy link

reverified ticket after deployment on test-de2 environment

@SahaLinaPrueger
Copy link
Author

@leventegal-she @AndyBakcsy-she Buttons INCOMING and OUTGOING are not yet translated on UI. Can you please check this?

@leventegal-she
Copy link
Contributor

@SahaLinaPrueger I pushed a fix for it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Affects the web backend change A change of an existing feature (ticket type) qa-verified Issue has been tested and verified by QA s2s Issues related to the SORMAS2SORMAS functionality vaadin-app Affects the Vaadin application
Projects
None yet
9 participants