Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(230)

Issue 54813003: remove community, redirect to support, clean up support (Closed)

Created:
7 years, 1 month ago by sethladd
Modified:
7 years, 1 month ago
Reviewers:
Kathy Walrath
Base URL:
git@github.com:dart-lang/dartlang.org.git@master
Visibility:
Public.

Description

remove community, redirect to support, clean up support BUG= R=kathyw@google.com Committed: 8ad8288

Patch Set 1 #

Total comments: 4

Patch Set 2 : tweaks #

Unified diffs Side-by-side diffs Delta from patch set Stats (+28 lines, -82 lines) Patch
M src/appengine/app.yaml View 1 chunk +3 lines, -0 lines 0 comments Download
M src/appengine/scripts/redirector.py View 1 chunk +2 lines, -0 lines 0 comments Download
M src/site/_includes/header.html View 1 chunk +1 line, -2 lines 0 comments Download
D src/site/community/index.html View 1 chunk +0 lines, -51 lines 0 comments Download
M src/site/index.html View 1 chunk +1 line, -2 lines 0 comments Download
M src/site/support/index.markdown View 1 3 chunks +21 lines, -27 lines 0 comments Download

Messages

Total messages: 9 (0 generated)
sethladd
ptal. thanks!
7 years, 1 month ago (2013-10-31 17:38:07 UTC) #1
Kathy Walrath
On 2013/10/31 17:38:07, sethladd wrote: > ptal. thanks! Could I look at this staged?
7 years, 1 month ago (2013-10-31 20:31:51 UTC) #2
sethladd
Sure: https://lists-dot-dart-lang.appspot.com/
7 years, 1 month ago (2013-10-31 20:42:29 UTC) #3
Kathy Walrath
I'm not crazy about hiding all the links in the support/ page. We have plenty ...
7 years, 1 month ago (2013-10-31 21:02:45 UTC) #4
sethladd
tweaked, thanks. Also, redeployed https://chromiumcodereview.appspot.com/54813003/diff/1/src/site/support/index.markdown File src/site/support/index.markdown (right): https://chromiumcodereview.appspot.com/54813003/diff/1/src/site/support/index.markdown#newcode4 src/site/support/index.markdown:4: description: "Mailing lists, Google+ communities, ...
7 years, 1 month ago (2013-10-31 21:08:50 UTC) #5
Kathy Walrath
lgtm One comment about linking to specific errors, but that's not required. https://chromiumcodereview.appspot.com/54813003/diff/1/src/site/support/index.markdown File src/site/support/index.markdown ...
7 years, 1 month ago (2013-10-31 22:08:21 UTC) #6
sethladd
Except when I go from that link to New issue it drops the area :(
7 years, 1 month ago (2013-10-31 22:10:05 UTC) #7
sethladd
Committed patchset #2 manually as r8ad8288 (presubmit successful).
7 years, 1 month ago (2013-10-31 22:10:24 UTC) #8
Kathy Walrath
7 years, 1 month ago (2013-10-31 22:21:11 UTC) #9
Message was sent while issue was closed.
On 2013/10/31 22:10:05, sethladd wrote:
> Except when I go from that link to New issue it drops the area :(

Yeah, you can read the bugs in the area (which is useful), but you can't file
one in a specific area unless we provide a link to a template. We might want to
provide templates for some areas (especially when there's specialized info to
request), but probably not for all of them.

Powered by Google App Engine
This is Rietveld 408576698