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

Stop termux-notification --button1-action previous execution overwrite #711

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Benjamin-Loison
Copy link

@Benjamin-Loison Benjamin-Loison commented Sep 19, 2024

3bea194#diff-343e04246aa27ee11598690b750ef754846d37f1107456982925ae4526689ce7 introduces the issue.

Last commit has the issue:

c553bc2ba7de37ad5c42df23d71e699d2785d7c5.mp4

While this pull requests solves the issue by restoring v0.50.1 behavior:

pull_request.mp4

Bash function used:

notification-test() {
    termux-notification -t "Test $1" --button1 "Button $1" --button1-action "echo Button $1 >> termux_notification_action.txt"
}
notification-test 0

Tested on Fairphone 4 LineageOS 21 with Termux app c2d57f2ed810b6fa854a6c9ab0ecee4aac38a0cc.

More details at Benjamin-Loison/termux-api/issues/23.

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.

1 participant