Closed ebanner closed 3 months ago
Closing because it seems unnecessary to add an endpoint since we're hitting the API pretty infrequently and we can just do the filtering on the ruby/client side
@ebanner I agree with your previous comment, but I do think that date-based filters would be a useful addition to the API.
That said, I would recommend a slightly modified approach to what was included in this PR-- rather than a single endpoint that can be only for a particular use case (events within the next hour), what I'd prefer to see would look like a pair of query parameter filters:
With these:
https://events.api.tampa.dev/?within_hours=1
https://events.api.tampa.dev/?within_days=14
This not only satisfies the original use case, but it also provides a pretty flexible utility filter that lets downstream consumers avoid doing at least some (if not most of) the date-based filtering themselves.
In order to have 60 minute event reminders (https://github.com/TampaDevs/events-slack-push/pull/2), this PR adds a
/next-hour
endpoint to show meetups happening in the next hour