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: mark as done on unsubscribe #1498

Merged
merged 3 commits into from
Sep 2, 2024
Merged

feat: mark as done on unsubscribe #1498

merged 3 commits into from
Sep 2, 2024

Conversation

setchy
Copy link
Member

@setchy setchy commented Aug 30, 2024

closes #1497

Screenshot 2024-08-30 at 8 16 20 AM

Signed-off-by: Adam Setch <adam.setch@outlook.com>
@github-actions github-actions bot added the enhancement New feature or enhancement to existing functionality label Aug 30, 2024
Signed-off-by: Adam Setch <adam.setch@outlook.com>
Signed-off-by: Adam Setch <adam.setch@outlook.com>
@setchy setchy marked this pull request as ready for review August 30, 2024 12:22
@setchy setchy added this to the Release 5.14.0 milestone Aug 30, 2024
Copy link
Collaborator

@bmulholland bmulholland left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why not do this by default?

@setchy
Copy link
Member Author

setchy commented Sep 2, 2024

Why not do this by default?

No real reason...

For comparison, "mark as done on open" is opt-in.

The good thing is we can always update the defaults in future

@setchy setchy merged commit 84f6fb2 into main Sep 2, 2024
12 of 13 checks passed
@setchy setchy deleted the feature/done-unsubscribe branch September 2, 2024 16:39
@bmulholland
Copy link
Collaborator

For comparison, "mark as done on open" is opt-in.

IIRC that was partly because Github usually does that itself. Could be wrong.

The thing that's even more so about this action is that unsubscribe is a write action already; mark as done seems usually implicit to me.

Just my 2c :)

@y-nk
Copy link

y-nk commented Sep 3, 2024

extremely grateful for this 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or enhancement to existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: allow mark as done on unsubscribe
3 participants