Unify endpoints into coherent API

Metro-Records / la-metro-councilmatic

:metro: An instance of councilmatic for LA Metro

MIT License

6 stars 2 forks source link

Unify endpoints into coherent API #635

Open hancush opened 4 years ago

hancush commented 4 years ago

We have a few endpoints for various actions:

/refresh-guid/ triggers syncs SES topics
/flush-cache/ flushes the cache
/public-comment/ redirects to the public comment page for the live meeting

I'm about to add another, /object-counts/, to report bill and event counts to scraping dashboard.

It would be ideal to have a unified API framework for these endpoints.

fgomez828 commented 3 years ago

Django's login_required documentation states that the decorator redirects users to a LOGIN_URL, which, if unspecified, is accounts/login?next=some/redirect/url.

~~We should find a way of removing this redirect since we don't use Django accounts.~~ We should find a way to avoid this redirect when a user is not authenticated.

hancush commented 3 years ago

@fatima3558 We do use Django accounts, but we hit these URLs programmatically, so the redirect when a user is not authenticated is a little awkward.