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

Unified Diff: includes/jobs/header.tmpl

Issue 29570571: Issue 5846 - Add support for greenhouse for jobs on eyeo.com (Closed) Base URL: https://hg.adblockplus.org/web.eyeo.com
Patch Set: Created Oct. 9, 2017, 11:49 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « includes/jobs/footer.tmpl ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: includes/jobs/header.tmpl
===================================================================
--- a/includes/jobs/header.tmpl
+++ b/includes/jobs/header.tmpl
@@ -7,43 +7,47 @@
<style>
#tabs-openings .ui-tabs-nav-vertical
{
font-size: 16px;
}
</style>
{% endblock %}
+{% set GREENHOUSE = 1 %}
+{% set TALENTTRACKER = 2 %}
+{% set EMAIL = 3 %}
ire 2017/10/10 08:18:33 NIT: I found this a bit confusing to understand at
juliandoucette 2017/10/10 10:46:22 - variable / constant names are more commonly auto
juliandoucette 2017/10/10 10:57:12 @wspee Do you think that this makes a difference t
ire 2017/10/10 11:43:52 We probably don't need the EMAIL constant anymore
ire 2017/10/10 11:43:52 Okay, I agree with your decision
+
{% set openings = [
- ("IT System Administrator", "it-system-administrator", 189),
- ("Data Engineer", "data-engineer", 194),
- ("Technical Support Analyst", "technical-support-analyst", 191),
- ("Filter List Supporter", "filter-list-supporter", 127),
- ("Whitelist Maintainer", "whitelist-maintainer", 108),
- ("Flattr Campaign Coordinator (Student Help)", "flattr-campaign-coordinator-student-help", 195),
- ("Student Help Acceptable Ads Monitoring", "student-help-acceptable-ads-monitoring", 193),
- ("Student Help Data Protection (m/w)", "student-help-data-protection", 178),
- ("Speculative Application", "speculative-application", 51),
+ ("IT System Administrator", "it-system-administrator", TALENTTRACKER, 189),
+ ("Data Engineer", "data-engineer", TALENTTRACKER, 194),
+ ("Technical Support Analyst", "technical-support-analyst", TALENTTRACKER, 191),
+ ("Filter List Supporter", "filter-list-supporter", TALENTTRACKER, 127),
+ ("Whitelist Maintainer", "whitelist-maintainer", TALENTTRACKER, 108),
+ ("Flattr Campaign Coordinator (Student Help)", "flattr-campaign-coordinator-student-help", TALENTTRACKER, 195),
+ ("Student Help Acceptable Ads Monitoring", "student-help-acceptable-ads-monitoring", TALENTTRACKER, 193),
+ ("Student Help Data Protection (m/w)", "student-help-data-protection", TALENTTRACKER, 178),
+ ("Speculative Application", "speculative-application", TALENTTRACKER, 51),
] %}
{% set opening = openings|find(page.split("/")[-1], attribute=1) %}
{% set title = (opening or [title])[0] %}
{% set og_image = "https://eyeo.com/images/eyeo-meta-jobs.png" %}
<header class="page-title">
<h1>Working at eyeo</h1>
</header>
<div id="jobs-info" class="content-block">
<div id="tabs-openings" class="columns-container one-sidebar">
<div class="column sidebar-left">
<h3 class="heading centered"><span>Current openings:</span></h3>
<ul class="ui-tabs-nav ui-tabs-nav-vertical">
- {% for title, file, _ in openings %}
+ {% for title, file, type, id in openings %}
<li {% if file == page.split("/")[-1] %} class="ui-tabs-active" {% endif %}>
{{("jobs/" + file)|linkify(class="ui-tabs-anchor")}}
{{ title }}
{% if file in german_openings %}
<img src="/images/flags/germany.png" alt="German" width="15" height="10">
{% endif %}
</a>
</li>
« no previous file with comments | « includes/jobs/footer.tmpl ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld