Skip to content

Draft: [BB-8204] feat: Detect Collapsed Items in Checklists

Stephannie Jimenez Gacha requested to merge detect-task-items into master

Short description on what are changed/implemented in this pull request

Supporting information

Discussions

Link to any public discussions about this PR or the design/architecture. Otherwise omit this.

Dependencies

None or link it here.

Visual changes

image

Merge deadline

None

Testing instructions

Step by step procedure

  1. Checkout to this branch
  2. Start the Listaflow docker
  3. Start the front-end development server
  4. Log in with admin@example.com
  5. Create a new list using Ghost White template
  6. Don't fill any required items and try to submit the form
  7. You should be scrolled to the first required item needed to submit
  8. All the missing required items should have the error message flagging it

Author notes and concerns

I have a couple of questions of weird behaviors with the current implementation,

  1. The subtasks inside a checklist isn't propagating a change to the taskController, which creates multiple side effects including:
  • The progress bar doesn't update
  • The error message will not dissapear
  • The main controller of the checklist doesn't report that the taskItem is completed

For this issue I checked out the states of all the controllers and found out that the cc controllers are updating correctly, but they are not synchronized with the main taskController meaning that we are loosing the reference of each child controller when useList is used to create the list of all the subtasks. At the end, it is never entering the useEffect that updates all controllers correctly and that's why none of the other elements in the page are updating correctly. I'm not sure what is the best approach to fix this, but once that is synchronized everything should be working as expected...

  1. Once the error messages appear, the user will not be able to submit the form. What is the expected behavior for this?
Edited by Stephannie Jimenez Gacha

Merge request reports