Skip to content
Snippets Groups Projects

move task_list setup and events to own class

Merged Simon Knox requested to merge task_list_refactor into master

What does this MR do?

deduplicate some task list code. Will need some more changes to work with Notes' task lists

Are there points in the code the reviewer needs to double check?

Clicking checkbox on a Note used to submit the form. This meant the note was marked as edited. Now it uses PATCH (same as issue/MR) so ticking boxes only updates description, and doesn't flag as edited

Also doesn't need a changelog, that was created mainly for science. Can delete

Why was this MR needed?

less code

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

#27246 (closed)

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading