Skip to content

Conversation

@VictorChan880
Copy link

@VictorChan880 VictorChan880 commented Feb 7, 2026

Notion ticket link

Edit Task Template Feature

Implementation description

  • Save saves template to database and redirects to Task Management Page
  • Delete confirmation modal appears upon clicking 'Delete Task'
  • Confirm delete actually deletes task
  • Success toast shown on save/delete
  • Error toast shown on failure and page does not navigate away

Steps to test

SQL for populating table:

INSERT INTO task_templates (task_name, category, instruction) VALUES
('Morning walk', 'Walk', 'Morning walks with the walker walkwalkwalkwalkwalkwalkwalkwalk…'),
('Everything is Awesome!', 'Games', 'The focus of Everything is Awesome game is to make the dog feel appreciated and engaged.'),
('Nap time', 'Pen Time', 'Instructions here blablablabalbblablablablablablalbalb.'),
('Feeding', 'Husbandry', 'Feeding the pet instructions here filler text filler text filler text filler text.'),
('Toilet training', 'Training', 'Step‐by‐step guide to toilet training your pet goes here.'),
('blahblahblahblahblahbalhalhalblahblabla…', 'Misc.', 'Instructions here blablablabalbblablablablablablablblabb.');

On Success Demo:
https://www.loom.com/share/d7058f5e4d76479bb16986fba6ee2cb9

On Error Demo:
https://www.loom.com/share/6e08d1ee314d494bb3d27e5226c58d85

What should reviewers focus on?

Though not apart of this ticket, I renamed the 'fetchUser' -> 'fetchTask' inside of the useEffect.

Checklist

  • My PR name is descriptive and in imperative tense
  • My commit messages are descriptive and in imperative tense. My commits are atomic and trivial commits are squashed or fixup'd into non-trivial commits
  • I have run the appropriate linter(s)
  • I have requested a review from the PL, as well as other devs who have background knowledge on this PR or who will be building on top of this PR

@VictorChan880 VictorChan880 self-assigned this Feb 7, 2026
@Akishai18 Akishai18 closed this Feb 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants