Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 29332920: Noissue - Removed compatibility for legacy job URLs (Closed)

Created:
Dec. 21, 2015, 1:47 p.m. by Sebastian Noack
Modified:
Dec. 21, 2015, 2:12 p.m.
Reviewers:
saroyanm
CC:
Felix Dahlke
Visibility:
Public.

Description

Noissue - Removed compatibility for legacy job URLs

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -4 lines) Patch
M pages/jobs/index.tmpl View 1 chunk +0 lines, -4 lines 0 comments Download

Messages

Total messages: 2
Sebastian Noack
I think we can remove that code now. For reference, initially we used a single ...
Dec. 21, 2015, 1:52 p.m. (2015-12-21 13:52:39 UTC) #1
saroyanm
Dec. 21, 2015, 1:57 p.m. (2015-12-21 13:57:50 UTC) #2
On 2015/12/21 13:52:39, Sebastian Noack wrote:
> I think we can remove that code now. For reference, initially we used a single
> page for our job ads, linking to them by fragment/hash. But for the past 9
> months every job ad has a separate page now (issue 662).

LGTM

Powered by Google App Engine
This is Rietveld