Skip to content

When I click on the notification, it is marked as read, but it doesn't open the GitHub PR like it used to #1550

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

Closed
1 task done
mrodriguez90 opened this issue Sep 25, 2024 · 7 comments · Fixed by #1552
Closed
1 task done
Labels
bug Something isn't working priority:low Items of low importance. Applicable to select few users or use-cases

Comments

@mrodriguez90
Copy link

mrodriguez90 commented Sep 25, 2024

🔍 Is there already an issue for your problem?

  • I have checked older issues, open and closed

📝 Description

When I click on the notification, it is marked as read, but it doesn't open the GitHub PR like it used to.
Maybe this isn't an issue, and it's a new setting I need to enable after the last version release?

🪜 Steps To Reproduce

  1. Open Gitify desktop
  2. Click on a PR notification

Gitify Version

5.14.0

Operating System

macOS

GitHub Account

Combination

📸 Screenshots

No response

@mrodriguez90 mrodriguez90 added the bug Something isn't working label Sep 25, 2024
@setchy
Copy link
Member

setchy commented Sep 25, 2024

@mrodriguez90 - does it open the notification at all?

@mrodriguez90
Copy link
Author

@mrodriguez90 - does it open the notification at all?

@setchy nope:

gitify.mp4

@setchy
Copy link
Member

setchy commented Sep 25, 2024

Interesting, thanks for the recording.

Is there anything noteworthy in the console log or app logs (see: https://www.gitify.io/faq/)?

@mrodriguez90
Copy link
Author

mrodriguez90 commented Sep 25, 2024

Interesting, thanks for the recording.

Is there anything noteworthy in the console log or app logs (see: https://www.gitify.io/faq/)?

@setchy yes, I found this:

[2024-09-25 12:57:49.459] [error] Error occurred while fetching notification html url AxiosError: Request failed with status code 404
    at xr (file:///Applications/Gitify.app/Contents/Resources/app.asar/build/js/app.js:60:17247)
    at XMLHttpRequest.A (file:///Applications/Gitify.app/Contents/Resources/app.asar/build/js/app.js:60:21926)
    at ra.request (file:///Applications/Gitify.app/Contents/Resources/app.asar/build/js/app.js:60:30099)
    at async ii (file:///Applications/Gitify.app/Contents/Resources/app.asar/build/js/app.js:60:47773)
    at async file:///Applications/Gitify.app/Contents/Resources/app.asar/build/js/app.js:60:65736
    at async _o (file:///Applications/Gitify.app/Contents/Resources/app.asar/build/js/app.js:60:65634)

BTW, I'm using Chrome Version 129.0.6668.70

@setchy
Copy link
Member

setchy commented Sep 25, 2024

This is helpful. My next suggestion would be (if possible) to run Gitify from source code and debug why the following section is throwing an exception.

gitify/src/utils/helpers.ts

Lines 115 to 130 in ae72183

export async function generateGitHubWebUrl(
notification: Notification,
): Promise<Link> {
const url = new URL(notification.repository.html_url);
if (notification.subject.latest_comment_url) {
url.href = await getHtmlUrl(
notification.subject.latest_comment_url,
notification.account.token,
);
} else if (notification.subject.url) {
url.href = await getHtmlUrl(
notification.subject.url,
notification.account.token,
);
} else {

@mrodriguez90
Copy link
Author

@setchy I solved this by signing out and then signing back in (I should have tried that earlier; my bad). It seems something happened with the repository permissions after updating Gitify.
Thanks for your help!

Side note: It was confusing that I could see notifications for all the repos, but the URL/redirect didn't work.

@setchy
Copy link
Member

setchy commented Sep 25, 2024

Glad to hear that you found a fix.

I'll also look to update this logic to have a "worst case" fallback for future

@setchy setchy added the priority:low Items of low importance. Applicable to select few users or use-cases label Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working priority:low Items of low importance. Applicable to select few users or use-cases
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants