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

Ability to Delete and Bulk Delete Transactions #832

Closed
4 tasks
zachgoll opened this issue Jun 4, 2024 · 0 comments · Fixed by #845
Closed
4 tasks

Ability to Delete and Bulk Delete Transactions #832

zachgoll opened this issue Jun 4, 2024 · 0 comments · Fixed by #845
Assignees

Comments

@zachgoll
Copy link
Collaborator

zachgoll commented Jun 4, 2024

Feature Overview

This feature allows users to delete financial transactions individually or in bulk, ensuring accurate and streamlined transaction management. Users can delete a single transaction via the transaction detail view or delete multiple transactions using the bulk edit bar.

Requirements

If there is a missing / incorrect requirement, please leave a comment before starting work on this.

A single transaction can be deleted by:

  • Opening the transaction detail view & deleting the transaction within the Settings section
  • Selecting the transaction, triggering the bulk edit bar and then choosing the Delete icon

Similarly, multiple transactions can also be deleted via:

  • The transactions detail view (which is triggered after selecting multiple txns and choosing Edit) within the Settings section
  • Selecting multiple transactions, triggering the bulk edit bar and choosing the Delete icon

Implementation Suggestions

Below are some ideas for implementation to get you started. Use your best judgment here—if there's a better way to do things, go for it!

Designs

Below are the designs you should follow while implementing this:

Flow Figma Link
Delete Single Transaction from Detail View https://www.figma.com/design/lonJmVk3HYkwZoIO7xYP2w/Maybe-App-(Community)?node-id=3611-6116
Delete Single Transaction via Bulk Edit Bar https://www.figma.com/design/lonJmVk3HYkwZoIO7xYP2w/Maybe-App-(Community)?node-id=3611-6117
Delete Multiple Transactions via Detail View https://www.figma.com/design/lonJmVk3HYkwZoIO7xYP2w/Maybe-App-(Community)?node-id=3611-6119
Delete Multiple Transactions via Bulk Edit Bar https://www.figma.com/design/lonJmVk3HYkwZoIO7xYP2w/Maybe-App-(Community)?node-id=3611-6118

If there is something in the mocks that you need more context around within

Reminders

  • Make sure to review our contributing guidelines before starting on an issue
  • We do our best to define a clear spec for new features and fixes, but think of them as "suggestions", not "hard requirements". We welcome ideas and suggestions!
    • If you see missing requirements to this issue, please leave a comment below explaining what is missing and why it is important.
    • If you see a requirement that you think is incorrect or not optimal, please leave a comment explaining what you think needs to change below.
@zachgoll zachgoll converted this from a draft issue Jun 4, 2024
@zachgoll zachgoll self-assigned this Jun 7, 2024
@zachgoll zachgoll moved this from Planning to In Progress in Transaction Management Jun 7, 2024
@zachgoll zachgoll linked a pull request Jun 7, 2024 that will close this issue
@zachgoll zachgoll moved this from In Progress to Done in Transaction Management Jun 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging a pull request may close this issue.

1 participant