@mnapoli opened this issue on April 9th 2015

Fixes #7620

Pull request based on #7578, don't merge this one before merging #7578.

Current design:

There's not much to say in that page so it feels a bit empty…

@mattab commented on April 9th 2015

the visual looks good to me, a solid improvement!

(I didn't look at code)

@mnapoli commented on April 9th 2015

FYI the diff is huge because this PR is based on the other PR which is also based on another PR :)

But the commit is very simple: 71c7a7d4d2f29634c621d5aaedaa8eee09880a01

@mnapoli commented on April 17th 2015

Keep in mind while reviewing that this PR is based on #7578, only the 2 last commits are about the maintenance.

@mattab commented on April 17th 2015
  • the subtitle we can use Piwik::getRandomTitle() instead of free/libre analytics platform
  • otherwise looks good +1 to merge
@mnapoli commented on April 17th 2015

It now uses Piwik::getRandomTitle()

@mnapoli commented on April 17th 2015

@mattab I'm afraid the commits are not good here :/ I was rebasing this PR because as you can see the diff includes some commits of the updater redesign, which is not good. I don't know if there are any consequences.

Only the 2 last commits should have been merged.

This issue was closed on April 17th 2015
Powered by GitHub Issue Mirror