-
Notifications
You must be signed in to change notification settings - Fork 31
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 REQ ] Auto clean on copy code #143
Comments
Not just on copy, but also after a certain amount of time. 2fa codes only work for so long, even if the text wasn't copied/read there's no value in keeping it after days. |
This would require quik messing with the clipboard, and we can't guarantee that you haven't copied something else in the meantime. to remove it after a certain amount of time would mean quik would have to read your clipboard to see if it's the same as what it copied, and then overwrite. that means storing the code for that duration, and being able to read whatever you copied between "copy code" and timer end. That sounds like an invasion of user privacy. As far as copy > read / archive / delete, let's talk about it more. There's a solution there somewhere. |
Yea, the clipboard thing I don't think is relevant, that was someone elses comment unrelated to the issue. If you've got a stale entry that won't work any way, who cares. But yes. An option to archive/delete/mark read/remove notification, on click of the copy button would be really handy. I use it several times a day, the annoyance of having to go back stacks up lol. I'd also really like to thank you for having the option to copy out the code from the notification in the first place. That is absolutely fantastic and a huge selling point for me. |
No, 2FA are time based, it doesn't matter if the code was copied or not in the clipboard, after say an hour there is no benefit of keeping the SMS anymore irrelevantly of if it was read or not as the code wouldn't work anyway. If you think of it that way, you don't need delete on copy anymore, since you have a garbage collector in place. Now for people with lots of SMSs it might still be much nicer to delete right away rather than waiting for the timer. |
Yea, that was sort of my point. There's no security concerns in keeping it in the clipboard, and empty clipboard VS a random number in it when the next thing you'll do with your clipboard is put something else in there is neither here nor there. Dev work for a feature that almost literally does nothing seems both a waste, and completely unrelated to this feature request. |
I'm not talking about the clipboard at all. I'm just saying at receive time + T delete the SMS automatically. Not sure where the confusion arises. It does what you asked for and more. |
Is your feature request related to a problem? If so, please describe the problem.
When an SMS with a 2fa code comes in, I am able to copy it directly from the notification. Which is wonderful. But I then still have to go back to mark it as read/archive it.
Describe the feature you'd like
The option to auto mark read sms when using the copy code function. Even better would be an additional option to mark read and send to archive when using the copy code option.
Additional context
N/A
The text was updated successfully, but these errors were encountered: