Skip to content

Adds notification when there’s a new release available - #98

Open
kennethormandy wants to merge 4 commits into
masterfrom
ko-notify
Open

Adds notification when there’s a new release available#98
kennethormandy wants to merge 4 commits into
masterfrom
ko-notify

Conversation

@kennethormandy

Copy link
Copy Markdown
Collaborator

screenshot 2015-05-07 10 32 59

Let people know when there’s a new release of Surge available using update-notifier. The default timing settings are left in place, so you’d only see a notification at most once each day you use Surge, and never until after you’d completed a deploy.

pr

@kennethormandy

Copy link
Copy Markdown
Collaborator Author

To follow up our previous discussion, the info on when to ask to update again is stored in ~/.config/configstore/update-notifier-surge.yml, along with any other global config for Insight, Update Notifier, and Yo.

The file itself will look something like:

optOut: false
lastUpdateCheck: 1431021283294

Conflicts:
	package.json
Comment thread lib/middleware/notify.js

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Reverse this if for clarity's sake. When the situation is special, do something special.

if (special && case) {
  takeSpecialAction();
}
// always take this action
next();

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants