Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue with JSON API for the jobs #1587

Closed
exoticDFT opened this issue Sep 26, 2024 · 0 comments · Fixed by #1588
Closed

Issue with JSON API for the jobs #1587

exoticDFT opened this issue Sep 26, 2024 · 0 comments · Fixed by #1588
Assignees
Labels
bug Something isn't working

Comments

@exoticDFT
Copy link
Member

It seems that in the code that generates our JSON API for the jobs can end up without a "," for the last job in the feed, or has an extra "," if the last job in our for loop is expired. One way to fix this seems to be always adding the "," for each job in the for loop. It partially seems like this was the expectation as the last element in our generated JSON is an empty object, i.e. "{}".

@exoticDFT exoticDFT added the bug Something isn't working label Sep 26, 2024
@exoticDFT exoticDFT self-assigned this Sep 26, 2024
exoticDFT added a commit to exoticDFT/usrse.github.io that referenced this issue Sep 26, 2024
- 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
crd477 added a commit that referenced this issue Dec 5, 2024
Issue #1587: Fixes a bug in the JSON API for jobs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant