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

Gitify does not send any notification from github.com if it is not able to access to GitHub Enterprise server. #423

Closed
pddg opened this issue Jun 28, 2020 · 6 comments · Fixed by #1419
Labels
enhancement New feature or enhancement to existing functionality help-wanted Extra attention is needed priority:medium Items of medium importance. Applicable to most users or use-cases

Comments

@pddg
Copy link

pddg commented Jun 28, 2020

Environment

  • macOS Catalina 10.15.5
  • Gitify v3.0.5

Expected behavior

Notification of github.com is appeared even if Gitify cannot reach GitHub Enterprise (GH:E) server (i.e. server downed, or disconnected from the networks of GH:E server...etc).

Actual behavior

In my environment, I use VPN to connect to GH:E server. Gitify doesn't send any notifications from github.com during disabling the VPN. If the VPN is enabled (Gitify becomes to be able to connect to GH:E server), Gitify send the notifications from both of github.com and GH:E.

@lilyball
Copy link

It doesn't just not send notifications, but it also replaces the entire UI with a "Something went wrong" page so I can't even check the notifications myself.

@bmulholland
Copy link
Collaborator

bmulholland commented Oct 4, 2023

Notifications and displaying the window both require information from the server. If it can't contact the server, it doesn't know what to display. Beyond that, none of the buttons/links would work. We could perhaps improve caching and display that, but that's a lot of work for a very minor benefit in a niche scenario. I don't think it's worth the work.

@pddg
Copy link
Author

pddg commented Oct 5, 2023

Github.com and GH:E server are independent entities, and failures can occur at each.
Not being able to display both when one is inaccessible is not the behavior expected by users, I think.

However, I do not know how difficult it would be to change this behavior, so I respect the maintainer's decision.

@bmulholland
Copy link
Collaborator

Oh, I misread 😅

Yeah that makes more sense. I think the requests could be more resilient for errors.

@setchy
Copy link
Member

setchy commented Feb 23, 2024

I think fetchNotifications will need to be tweaked to support this.

const fetchNotifications = useCallback(

@pddg - contributions welcomed 🙏

@setchy setchy added enhancement New feature or enhancement to existing functionality help-wanted Extra attention is needed and removed feature-request labels Feb 23, 2024
@afonsojramos
Copy link
Member

This also happens in the case that a specific organisation requires SAML Authorisation.

image

@setchy setchy added the priority:medium Items of medium importance. Applicable to most users or use-cases label Jun 17, 2024
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 help-wanted Extra attention is needed priority:medium Items of medium importance. Applicable to most users or use-cases
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants