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

feat(DeleteItem): Add deletion of goals, microgoals, and tasks #18

Merged
merged 3 commits into from
Nov 4, 2024

Conversation

marycaserio
Copy link
Contributor

@marycaserio marycaserio commented Nov 4, 2024

  • Refactor: Merge all deletion logic inside the hook are all merged into single function deleteItem
  • Feat: Add new component to enable delete goals, microgoals, and tasks by just single click on the delete icon (with mistake handling).
  • Feat: Add new common components ConfirmationDialog.

Copy link

github-actions bot commented Nov 4, 2024

🚀 Preview for this PR is available at: https://stepwise-bdab7--pr-18-8rw95x1t.web.app

🧪 Test Results

🗂 Test Summary: Test Files 2 passed (2)
Tests Passed: Tests 4 passed (4)
⏱️ Total Duration: Duration 16.29s (transform 7.24s, setup 0ms, collect 30.78s, tests 392ms, environment 834ms, prepare 148ms)

🎉 All tests passed successfully!

@ZL-Asica ZL-Asica changed the title feat(DeleteItem, MacroGoal, MicroGoal, Task): deletion of goals, microgoals, and tasks feat(DeleteItem): Add deletion of goals, microgoals, and tasks Nov 4, 2024
Copy link

github-actions bot commented Nov 4, 2024

🚀 Preview for this PR is available at: https://stepwise-bdab7--pr-18-8rw95x1t.web.app

🧪 Test Results

🗂 Test Summary: Test Files 2 passed (2)
Tests Passed: Tests 4 passed (4)
⏱️ Total Duration: Duration 16.21s (transform 6.94s, setup 0ms, collect 30.69s, tests 327ms, environment 798ms, prepare 176ms)

🎉 All tests passed successfully!

@ZL-Asica ZL-Asica enabled auto-merge (squash) November 4, 2024 05:01
@ZL-Asica ZL-Asica merged commit 4e31a55 into main Nov 4, 2024
3 checks passed
@ZL-Asica ZL-Asica deleted the Deletion branch November 7, 2024 17:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants