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

fix assignments example #48

Merged
merged 1 commit into from
Aug 12, 2024
Merged

fix assignments example #48

merged 1 commit into from
Aug 12, 2024

Conversation

scottnicolson
Copy link

A user pointed out the following via support on Front.

I think there's a mistake in the API example Assignments Available for Review in Two Hours.
The query parameter used in that example "available_after=<2 hours from now>" would return the set of assignments that will not be available in two hours but will become available at some time after that.
To return all assignments that will be available by two hours from now (including assignments that are already available now), the query parameter should be "available_before=<2 hours from now>".
If the intent is to exclude assignments that are available now and return only those assignments that will become available in the next two hours, the query parameters should be "available_after=&available_before=<2 hours from now>".
Apologies if I've misunderstood something.

I am inclined to agree so I have updated the docs.


The credentials to view the Netlify deploy is the following:

  • username: crabigator
  • password: noblowfishallowed

@scottnicolson scottnicolson self-assigned this Aug 12, 2024
@scottnicolson scottnicolson changed the title fix timestamp fix assignments example Aug 12, 2024
@scottnicolson scottnicolson merged commit 1b7d04c into master Aug 12, 2024
4 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant