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

Complete status not showing after user adds mandatory fields for Result element #1337

Open
2 of 3 tasks
praweshsth opened this issue Jan 11, 2024 · 4 comments
Open
2 of 3 tasks
Assignees
Labels
Bug Inconsistencies or issues which will cause an issue or problem ODS Issue initiated by ODS

Comments

@praweshsth
Copy link
Collaborator

praweshsth commented Jan 11, 2024

Overview

Currently, when user adds all the mandatory data in result, the status of result is not showing completed. Need to fix it.

QA

  • Complete status of result shown in sidebar menu of activity elements
  • Complete status of result shown in summary page of the activity
  • Script to change the status of the result of existing activities in the production if all the mandatory data fields are completed.

Steps to replicate

  1. Fill the mandatory field of result element and indicator level
  2. Save the form .

Actual Output:
The result element status is displayed as incomplete, and as well as  no tick mark in the sidebar result element.
Image

Expected Output:
The status of the result element on the activity summary page should be presented as complete, and the tick mark should also be displayed in the result element in the sidebar.

@praweshsth praweshsth added the Bug Inconsistencies or issues which will cause an issue or problem label Jan 11, 2024
@praweshsth praweshsth changed the title As an organisation, I want to see the result is completed after adding mandatory data in it so that there is no confusion Complete status not showing after user adds mandatory fields for Result element Jan 11, 2024
@praweshsth
Copy link
Collaborator Author

@odscrachel I'm adding you here for you to track the issue.

@sarinasindurakar
Copy link
Collaborator

sarinasindurakar commented Jan 16, 2024

  • When there are multiple complete results, the addition of other incomplete result will still indicate an overall status as complete. The status does not get changed when the user modifies the results.
    image

@dan-odsc dan-odsc added the ODS Issue initiated by ODS label Jan 16, 2024
@praweshsth
Copy link
Collaborator Author

@odscrachel

If the fields of the Result as mentioned in this document sheet (Result only), which are mandatory, should be added to make the status of the Result 'Complete'. However, if there are multiple Result, all the Result's field as mentioned above should be added to make the Result's status 'Complete'.

Going through the fields, we need to discuss the logic to make Result's Status 'Complete' might need a change. Because some of the sub-elements or Result/ indicator such as 'Document Link' are not required but if a field within the sub-element is added then the required fields of the sub-element should be completed. In case if such sub elements' fields are not added, it shouldn't contribute to the status of the result to be complete. This logic is not implemented as of now. The logic searches all the fields marked as mandatory (in the document) are filled or not. So for now, user need to add all the mandatory fields to make the 'Result' Complete.

As per the IATI Standard, user should only add 'Reference' in either in Result level or in Indicator level but not in both. That is how the IATI Publisher is built. Since there are old data that contains 'Reference' in both levels, we will write the script to bypass the no-Reference-in-both-levels rule and make the Results' status complete if all the mandatory fields are completed as mentioned above in first paragraph. However, if user goes to edit the result or indicator, they will see warning message that Reference is already in another level. As I have mentioned earlier, one of the solutions to avoid the warning is to remove Reference from one level. But, as you have mentioned, we have to discuss it with the clients.

@praweshsth
Copy link
Collaborator Author

@A4family @sarinasindurakar In discussion with ODSC team yesterday, for now, let's bypass the activities that have reference in both levels (result and indicator) and show the results complete or not based on the required fields.

cc: @odscrachel

@praweshsth praweshsth assigned odscrachel and unassigned A4family Jan 24, 2024
@odscrachel odscrachel assigned praweshsth and unassigned odscrachel Jan 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Inconsistencies or issues which will cause an issue or problem ODS Issue initiated by ODS
Projects
None yet
Development

No branches or pull requests

5 participants