Skip to content

Add possibility to mark a thread/notification as done#2985

Merged
EnricoMi merged 2 commits intoPyGithub:mainfrom
m42e:feature/mark_notification_done
Jul 30, 2024
Merged

Add possibility to mark a thread/notification as done#2985
EnricoMi merged 2 commits intoPyGithub:mainfrom
m42e:feature/mark_notification_done

Conversation

@m42e
Copy link
Copy Markdown
Contributor

@m42e m42e commented Jun 6, 2024

This PR adds the ability to mark a notification as done, following the already implemented mark_as_read by #932. Duplicated by #2976.

Fixes #2947

Co-authored-by: Hodei Navarro hodei.navarro@outlook.com

A new function `mark_as_done` besides `mark_as_read`.
@m42e m42e force-pushed the feature/mark_notification_done branch from ce0d4df to 1a4b7cb Compare June 6, 2024 04:53
@m42e m42e changed the title Add possibility to mark a thread as done Add possibility to mark a thread/notification as done Jun 6, 2024
Copy link
Copy Markdown
Collaborator

@EnricoMi EnricoMi left a comment

Choose a reason for hiding this comment

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

Can you add a test for this new method, please?

@m42e
Copy link
Copy Markdown
Contributor Author

m42e commented Jul 30, 2024

@EnricoMi done

@m42e m42e force-pushed the feature/mark_notification_done branch 2 times, most recently from 4e0eb1c to 102d971 Compare July 30, 2024 03:49
@m42e m42e force-pushed the feature/mark_notification_done branch from 102d971 to cc3531e Compare July 30, 2024 03:51
Copy link
Copy Markdown
Collaborator

@EnricoMi EnricoMi left a comment

Choose a reason for hiding this comment

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

Awesome, LGTM!

@EnricoMi EnricoMi added this pull request to the merge queue Jul 30, 2024
Merged via the queue into PyGithub:main with commit 5ba2437 Jul 30, 2024
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.

mark notification as done

2 participants