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

[DEA] Location concept support issue #1316

Closed
Tracked by #1465
mahalakshme opened this issue Aug 27, 2024 · 7 comments
Closed
Tracked by #1465

[DEA] Location concept support issue #1316

mahalakshme opened this issue Aug 27, 2024 · 7 comments
Assignees

Comments

@mahalakshme
Copy link
Contributor

mahalakshme commented Aug 27, 2024

https://avni.freshdesk.com/a/tickets/4364

Need:

  • Seems like when location of lower location type is not known, higher location type will be set for the org in focus.
  • To handle the already (incorrectly)set values, independent of the implementation issue.

Issue:

No support in DEA LocationFormElement for Location type concept with 'highestAddressLevelTypeUUID' set. Only levels of 'lowestAddressLevelTypeUuids' are accepted and the component breaks when handling a level of a higher type.

Steps to reproduce:

org: IPH Sickle cell
search Participant 'Thanya' -> Profile-> Registration-> edit -> next -> you will get the below error page

Image

Out of scope:

Implementation issue.

@mahalakshme mahalakshme converted this from a draft issue Aug 27, 2024
@mahalakshme mahalakshme moved this from In Analysis to Ready in Avni Product Aug 27, 2024
@petmongrels petmongrels self-assigned this Aug 30, 2024
petmongrels added a commit that referenced this issue Aug 30, 2024
…form element.

UI changes to concept show to avoid truncation of long custom attribute names and uuid values.
@petmongrels petmongrels moved this from In Progress to Code Review Ready in Avni Product Aug 30, 2024
@petmongrels
Copy link
Contributor

I noticed there is a bug in form element status rule which is not allowing edit of the location field.

@mahalakshme
Copy link
Contributor Author

@petmongrels the above was communicated to the support team by Joy as part of the ticket. They will check with implementation team and resolve it. I informed the impl., team as well.

@1t5j0y 1t5j0y moved this from Code Review Ready to In Code Review in Avni Product Sep 2, 2024
@1t5j0y 1t5j0y moved this from In Code Review to QA Ready in Avni Product Sep 2, 2024
@Dinesh2019
Copy link

The value is not appearing in the location field after selection in the location concept

Reference video :

Click here to watch the video

@Dinesh2019 Dinesh2019 self-assigned this Sep 26, 2024
@Dinesh2019 Dinesh2019 moved this from In QA to QA Failed in Avni Product Sep 27, 2024
@AchalaBelokar
Copy link

@petmongrels
Copy link
Contributor

Registration form is not opening in this org. please test in some org which is correctly setup.

@petmongrels petmongrels moved this from In Progress to QA Ready in Avni Product Oct 1, 2024
@dinesh2096
Copy link

Web app is getting is crashed

link i used to for deploy the webapp

Click here to watch the video

@dinesh2096 dinesh2096 moved this from In QA to QA Failed in Avni Product Oct 16, 2024
@himeshr himeshr self-assigned this Oct 17, 2024
himeshr added a commit that referenced this issue Oct 17, 2024
…ed in Location FormElement when multiple lowestAddressLevelTypeUUIDs are set
@himeshr
Copy link
Contributor

himeshr commented Oct 17, 2024

Test cases

DEA, Within form, Location Form element

  • Hierarchial mode

    • Single lowerAddressLevelTypes
    • Multiple lowerAddressLevelTypes
    • With HigherAddressLevelType set
    • Without HigherAddressLevelType set
  • Regular search mode

    • Single lowerAddressLevelTypes
    • Multiple lowerAddressLevelTypes
    • With HigherAddressLevelType set
    • Without HigherAddressLevelType set

App designer

    • Create and edit of Location concept with Multiple lowerAddressLevelTypes, old setting should be retained

Rules specified for Location Form Element

  • Ensure that the rule written makes use of titleLineage(DEA) and locationMapping(Client) both, so that it works on DEA as well as Avni-Client
    Refer to Update Goonj rules to work in DEA #1252 for example.

Known issue

  • Unselect of Location is not supported in webapp

@himeshr himeshr moved this from In Progress to Code Review Ready in Avni Product Oct 17, 2024
@1t5j0y 1t5j0y moved this from Code Review Ready to In Code Review in Avni Product Oct 18, 2024
@1t5j0y 1t5j0y moved this from In Code Review to QA Ready in Avni Product Oct 18, 2024
@AchalaBelokar AchalaBelokar moved this from In QA to Done in Avni Product Oct 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

No branches or pull requests

7 participants