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

Do not input Level of Location type from user - deduce it #1309

Open
Tracked by #1656
mahalakshme opened this issue Aug 19, 2024 · 0 comments
Open
Tracked by #1656

Do not input Level of Location type from user - deduce it #1309

mahalakshme opened this issue Aug 19, 2024 · 0 comments

Comments

@mahalakshme
Copy link
Contributor

mahalakshme commented Aug 19, 2024

Need:

The level in Location type is confusing for the user to setup. And it is redundant considering the user has to setup Parent for each Location type.

AC:

  • The UI to be displayed similar to how we add concept answers
Screenshot 2024-09-18 at 2 41 59 PM
  • Show list:
    - On clicking 'Location Hierarchy' show the hierarchies
    - Should be able to Edit and Delete an hierarchy
    - It is okay if we lose the Export functionality doesn't work (since it was working using React admin - now we cant use it probably)

  • Create:

    • Location type at the top will be higher in the hierarchy(larger location type) and lower ones in the botton
    • Each entry should create a location type
    • Should be able to move location types up and down
    • On trying to enter name of a location type, the text need to change to 'Next smaller location type'
    • Should be able to select an already created location type from the drop down
Screenshot 2024-09-18 at 2 56 46 PM

Old : ignore

AC:

  • Do not input Level when creating Location type
  • Determine it based on parent entered.
    - Scenario: Create Location type A - level: 0
    - create type B with parent A - level of B: -1
    - create C, not set parent - level: 0 (whenever parent not set - level is 0)
    - edit A and set C as parent - update level of C to 1
  • Do not display level in Location type page as well

Technical analysis:

Looks like we are using level in mobile app at many places - to find max and to determine the display order.

Inputs:

  • start with something else instead of 0
  • set the entire hierarchy intuituvely
  • extremely low priority
  • ability to do vs ease of doing - this only is ease of doing.
  • edit of location hierarchy - shud be possible.
  • has come quite a bit.
  • Edit
  • not intuitive - if higher to lower or lower to higher.
@mahalakshme mahalakshme converted this from a draft issue Aug 19, 2024
@mahalakshme mahalakshme changed the title Auto-determine level when creating location type Deprecate level of Location type Aug 19, 2024
@mahalakshme mahalakshme moved this from In Analysis to In Analysis Review in Avni Product Sep 16, 2024
@mahalakshme mahalakshme moved this from In Analysis Review to Ready in Avni Product Sep 17, 2024
@mahalakshme mahalakshme changed the title Deprecate level of Location type Do not input Level of Location type from user - deduce it Sep 17, 2024
@mahalakshme mahalakshme moved this from Ready to In Analysis in Avni Product Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Triaged
Development

No branches or pull requests

1 participant