-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Delete card transactions when "allow deleting transactions" is disabled on card feed #54389
Comments
Triggered auto assignment to @kadiealexander ( |
Just noting that I'll be OOO until Jan 6th, not reassigning given most people will be out but please tag someone online from the bug zero team if any urgent action is needed! |
I would love to help on this issue as C |
@DylanDylann not sure if there is much to do for you here |
@mountiny, @kadiealexander, @DylanDylann Whoops! This issue is 2 days overdue. Let's get this updated quick! |
Havent got around yet |
Context
Third-party workspace card feeds in NewDot have an "allow deleting transactions" toggle on the feed settings. You should not be able to delete any card transactions imported while it was deleted. Similarly, you should be able to delete any transactions imported while it is enabled. The rule only applies to new transactions, eg: if you import transactions while this is disabled and later enable it, those past transactions can't be deleted and vice versa.
Problem
As discussed here, you can currently delete card transactions imported when this toggle is disabled, counter to how this feature is supposed to work.
2024-12-19_13-52-37.mp4
Solution
Fix the behavior of this feature for both custom and direct feeds such that it functions as originally described in the design doc.
Issue Owner
Current Issue Owner: @mountinyThe text was updated successfully, but these errors were encountered: