Skip to content

Amanda/delete task template#180

Merged
Akishai18 merged 2 commits intomainfrom
amanda/delete-task-template
Feb 10, 2026
Merged

Amanda/delete task template#180
Akishai18 merged 2 commits intomainfrom
amanda/delete-task-template

Conversation

@amanduhhhh
Copy link
Contributor

Notion ticket link

Create Delete Task Template Screen

Implementation description

  • Created delete modal based on PopupModal component
  • Connected the delete button to TaskTemplateAPIClient.deleteTaskTemplate

Steps to test

  1. Add a task template to your database
  2. Navigate to /admin/tasks/edit/{:your task template id}
  3. Delete the task template
image

What should reviewers focus on?

  • Ensure the delete modal looks fine (according to the figma)
  • Ensure delete button deletes your task properly
  • Side note: the edit task template page has some user placeholder terms, this was addressed in this pr

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

@amanduhhhh amanduhhhh requested a review from Akishai18 February 7, 2026 01:39
@amanduhhhh amanduhhhh force-pushed the amanda/delete-task-template branch from aa13c55 to 431cde3 Compare February 7, 2026 12:37
@amanduhhhh amanduhhhh force-pushed the amanda/delete-task-template branch from 431cde3 to 6a56c88 Compare February 9, 2026 20:17
Copy link
Collaborator

@Akishai18 Akishai18 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, Good work!

@Akishai18 Akishai18 added this pull request to the merge queue Feb 10, 2026
Merged via the queue into main with commit 0dbdc2b Feb 10, 2026
1 check passed
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