Skip to content

Commit

Permalink
Issue USRSE#1587: Fixes a bug in the JSON API for jobs
Browse files Browse the repository at this point in the history
- Adds a `,` for every element in our jobs array
- Updates formatting to see the liquid logic more clearly
- Moves nowunix capture outside of the for loop for jobs
  • Loading branch information
exoticDFT committed Sep 26, 2024
1 parent bec1961 commit eac9b40
Showing 1 changed file with 17 additions and 5 deletions.
22 changes: 17 additions & 5 deletions pages/api/jobs.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,22 @@ layout: none
permalink: /api/jobs.json
excluded_in_search: true
---
{% assign sorted_jobs = site.data.jobs | sort: "posted" | reverse %}[{% for job in sorted_jobs %}{% capture nowunix %}{{'now' | date: '%s'}}{% endcapture %}{% capture expires %}{{ job.expires | date: '%s'}}{% endcapture %}{% capture posted %}{{ job.posted | date: '%b %d, %Y'}}{% endcapture %}{% if expires > nowunix %}{
{%- assign sorted_jobs = site.data.jobs | sort: "posted" | reverse -%}
{%- capture nowunix -%}{{'now' | date: '%s'}}{%- endcapture -%}
[
{%- for job in sorted_jobs -%}
{%- capture expires -%}{{ job.expires | date: '%s'}}{%- endcapture -%}
{%- capture posted -%}{{ job.posted | date: '%b %d, %Y'}}{%- endcapture -%}
{% if expires > nowunix %}
{
"title": {{ job.name | jsonify }},
{% if posted != '' %}"published": "{{ posted }}",{% endif %}
"location": {{ job.location| jsonify }},
{%- if posted != '' %}
"published": "{{ posted }}",
{%- endif %}
"location": {{ job.location | jsonify }},
"url": "{{ job.url }}"
}{% if forloop.last %}{% else %},{% endif %}{% endif %}
{% endfor %}{}]
},
{%- endif %}
{%- endfor %}
{}
]

0 comments on commit eac9b40

Please sign in to comment.