Skip to content

feat: add PR update notifications - #33

Open
kaijianding wants to merge 1 commit into
SeanMcTex:mainfrom
kaijianding:pr_update
Open

feat: add PR update notifications#33
kaijianding wants to merge 1 commit into
SeanMcTex:mainfrom
kaijianding:pr_update

Conversation

@kaijianding

Copy link
Copy Markdown
Contributor

implement #25

and fix hide inactive pr preference doesn't work

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@kaijianding
kaijianding requested a review from SeanMcTex as a code owner June 8, 2026 09:25

@SeanMcTex SeanMcTex left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Hey @kaijianding -- sorry for the delay on this. Looking at the PR diff, it seems as though there's a lot of spurious info included in the PR. Could you please rebase this or create a new branch and cherry pick the relevant work so that I can dig into just the code that's actually part of this change? Thanks!

@kaijianding

kaijianding commented Jul 10, 2026

Copy link
Copy Markdown
Contributor Author

Hey @kaijianding -- sorry for the delay on this. Looking at the PR diff, it seems as though there's a lot of spurious info included in the PR. Could you please rebase this or create a new branch and cherry pick the relevant work so that I can dig into just the code that's actually part of this change? Thanks!

There are 3 parts

  1. update check itself
  2. add notification text in setting
  3. register notification

Do you mean remove the second and third part, remain only the first part? @SeanMcTex

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.

2 participants