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

Unified Diff: includes/jobs/footer.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: Addressed comments Created Oct. 10, 2017, 10:52 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 | « no previous file | includes/jobs/header.tmpl » ('j') | includes/jobs/header.tmpl » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: includes/jobs/footer.tmpl
===================================================================
--- a/includes/jobs/footer.tmpl
+++ b/includes/jobs/footer.tmpl
@@ -1,14 +1,17 @@
{% if opening %}
- <a href="
- {%- if opening[2] -%}
- https://app.talentstracker.com/job_applications/new?position_id={{ opening[2] }}
+ {% set tracker = opening[2] %}
+ <a class="button" href="
+ {%- if tracker == GREENHOUSE -%}
+ https://boards.greenhouse.io/eyeo/jobs/{{ opening[3] }}
+ {%- elif tracker == TALENTTRACKER -%}
+ https://app.talentstracker.com/job_applications/new?position_id={{ opening[3] }}
{%- else -%}
mailto:jobs@eyeo.com
{%- endif -%}
- "><button>apply now</button></a>
- {{ "jobs"|linkify }}<button class="why-work-here-button">why work here?</button></a>
+ ">apply now</a>
+ {{ "jobs"|linkify(class='button why-work-here-button') }}why work here?</a>
{% endif %}
</div>
</div>
</div>
</div>
« no previous file with comments | « no previous file | includes/jobs/header.tmpl » ('j') | includes/jobs/header.tmpl » ('J')

Powered by Google App Engine
This is Rietveld