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

Hide update nag #601

Merged
merged 1 commit into from
May 2, 2017
Merged

Hide update nag #601

merged 1 commit into from
May 2, 2017

Conversation

jasmussen
Copy link
Contributor

This PR hides the update nag. In part it's to open the discussion, in part it's because the update nag is already visible on every other screen in the admin, and it also has a spot in the adminbar even in the editor.

screen shot 2017-05-02 at 10 55 08

We will probably encounter further notice work in the future. There's a separate ticket for it (#486). But for now, let's see if we can't make an exception to the update nag on this screen.

This PR hides the update nag. In part it's to open the discussion, in part it's because the update nag is already visible on every other screen in the admin, and it also has a spot in the adminbar even in the editor.

We will probably encounter further notice work in the future. There's a separate ticket for it. But for now, let's see if we can't make an exception to the update nag on this screen.
@jasmussen jasmussen self-assigned this May 2, 2017
@jasmussen jasmussen requested a review from youknowriad May 2, 2017 08:57
Copy link
Contributor

@youknowriad youknowriad left a comment

Choose a reason for hiding this comment

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

Ok for me 👍

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