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

increase notification timeout up to 30m #913

Merged
merged 1 commit into from
Oct 22, 2016

Conversation

iamstarkov
Copy link
Contributor

fetchNotifications error can happen in two cases:

In both cases it doesn't hurt to wait half an hour.

PS. right now it hurts, when you are offline and developing
anything for Hyper, console.error logs the same error
every 10s and ruins your investigation.

fetchNotifications error can happen in two cases:
* https://hyper-news.now.sh is down
* you are offline

In both cases it doesn't hurt to wait half an hour.

PS. right now it hurts, when you are offline and developing
anything for Hyper, `console.error` logs the same error
every 10s and ruins your investigation.
@rauchg rauchg merged commit 3f77231 into vercel:master Oct 22, 2016
@iamstarkov iamstarkov deleted the fix/increase-notification-timeout branch October 22, 2016 19:38
@iamstarkov
Copy link
Contributor Author

yeah 🎉

chabou added a commit to chabou/hyper that referenced this pull request Nov 2, 2016
* master:
  chore(package): update electron to version 1.4.5 (vercel#949)
  add onRendererWindow, because it actually available. check ./lib/utils/plugins.js:193 (vercel#924)
  increase timeout for update checks (vercel#928)
  Change URL regex - fixes vercel#867 (vercel#943)
  Make the close menu items less ambiguous
  Comply to XO's no-warning-comments rule
  Prefer default export to make XO happy (vercel#931)
  chore(package): update ms to version 0.7.2 (vercel#933)
  chore(package): update copy-webpack-plugin to version 4.0.0 (vercel#927)
  log plugins' errors in Electron console (vercel#923)
  Improve tabs title   (vercel#892)
  increase notification timeout up to 30m (vercel#913)
  Create github templates (vercel#919)
  chore(package): update electron to version 1.4.4 (vercel#907)
  Prevent Hterm to share same preferences between instances (vercel#906)
  0.8.3
  0.9.0
  Revert "Quick full screen (vercel#803)"
  Ignore file watch errors. Fixes vercelGH-225 (vercel#893)
chabou added a commit to chabou/hyper that referenced this pull request Nov 2, 2016
* master:
  chore(package): update electron to version 1.4.5 (vercel#949)
  add onRendererWindow, because it actually available. check ./lib/utils/plugins.js:193 (vercel#924)
  increase timeout for update checks (vercel#928)
  Change URL regex - fixes vercel#867 (vercel#943)
  Make the close menu items less ambiguous
  Comply to XO's no-warning-comments rule
  Prefer default export to make XO happy (vercel#931)
  chore(package): update ms to version 0.7.2 (vercel#933)
  chore(package): update copy-webpack-plugin to version 4.0.0 (vercel#927)
  log plugins' errors in Electron console (vercel#923)
  Improve tabs title   (vercel#892)
  increase notification timeout up to 30m (vercel#913)
  Create github templates (vercel#919)
  chore(package): update electron to version 1.4.4 (vercel#907)
tylerong pushed a commit to tylerong/hyper that referenced this pull request Jul 3, 2017
fetchNotifications error can happen in two cases:
* https://hyper-news.now.sh is down
* you are offline

In both cases it doesn't hurt to wait half an hour.

PS. right now it hurts, when you are offline and developing
anything for Hyper, `console.error` logs the same error
every 10s and ruins your investigation.
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.

None yet

2 participants