-
Notifications
You must be signed in to change notification settings - Fork 6
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
[Drupal] MVP API #18
Comments
@AnthonyAstige do you know if we will want to filter by a date range or maybe, to keep it simple, just individual dates. For example, the consumer side could send multiple requests, one for each date it needs data for. This is just a thought, just let me know what you think you will be looking for |
@Lowell20 Just one date at a time sounds good for the MVP. @aaronsalmon - See Lowell's prior comment, in case you're wire framing up something we may have a need for a date range search soon. |
@Lowell20 - The api gives dates like "2016-05-04T01:30:00". It looks like UTC, but we think UTC usually has a Z at the end. We need to be sure we're doing the right stuff with times/timezones. Can you confirm it's a standard time format and let us know how to identify the timezone, and/or patch it? |
restructured a few things, here is the beginning of the endpoints naming, including a vendor endpoint |
I can confirm it is UTC time |
What's the difference between these: |
I think my goal is to filter the results by a date or date range, the "date-or-timestamp" text can be replaced with the query parameters. |
For the MVP we would need API endpoints for:
I'm no too particular on how you want to format the API so long as we can get access to the data. |
RE: 2. Simpler and more clear I think to just have the date with no time in the url imo. ie; RE: 3. I'm not sure we need single event query in the API for the MVP - #30 can easily retrieve by date and filter locally by Event UUID. (#38 for lodash opened to help here) |
We have enough in the API now I think for MVP. Issues will fall out while coding as needed from here. |
The [Consumer] side needs this data / api to work
The text was updated successfully, but these errors were encountered: