Closed brianfeister closed 1 month ago
Yeeeeee HAWWW !!! Self merging here because this is almost "un reviewable" due to the simple fact that the total changes are over 3K lines and there's not time to review
This is SUPER CRUNCH time and relying on test coverage is probably our only hope (which is similar to self-review)
I've bumped test coverage up to 55% with this PR
Putting this up for early review.
What does this PR do
SeshuSessions
at minimum, so we don't need to entirely remove Dynamo as we're using it)X-Mnm-Subdomain-Value
header as marqo search filter on home page search, so https://brians-pub.testingnear.me/?lat=39.8616981506&lon=-104.672996521&radius=20500 will show only events owned by the user ID associated withbrians-pub
subdomain. Compare to https://testingnear.me/?lat=39.8616981506&lon=-104.672996521&radius=20500 which should show 8 events{"/api/event", "POST", handlers.PostEventHandler, None},
{"/api/events", "POST", handlers.PostBatchEventsHandler, None},
{"/api/events", "GET", handlers.SearchEventsHandler, None},
{"/api/events/{eventId}", "GET", handlers.PostEventHandler, None},
radius
param values here indicate there's a problem with the calculation math TODO: fix this