search

whoownsphilly / wop-web

https://wop-web.vercel.app
2 stars 1 forks source link

This takes too long: "Finding all owner timeline related information (may take some time)..." #113

Closed dcaud closed 2 years ago

dcaud commented 3 years ago

It was taking approx. 2 to 10 seconds depending on what property I clicked on.

It looks like querying the timeline info takes a lot of time. I think it should be removed for the MVP. Then added back in later when enough energy can be put forth to make it not slow down the main query.

ssuffian commented 3 years ago

The timeline is how it figures out what owners are connected and who owns what when, its required for things like gathering violations and complaints by owner. Otherwise all the information is historical data by property rather than by owner. Without it, we could show other properties currently connected to that owner and a history of violations/complaints at the selected property, but anything beyond that requires the who owns-what-when part.

I can look into moving it to the owner tab so that its a secondary loading thing rather than necessary for the Summary page.

dcaud commented 3 years ago

I misunderstood what it meant: I thought it was loading the data for the (cool) new history graph you previewed yesterday. Now that I understand what it is, I don’t think it can be easily changed. It should probably just be left as is!

On Oct 7, 2021, at 10:44 PM, Stephen Suffian @.***> wrote:

 The timeline is how it figures out what owners are connected and who owns what when, its required for things like gathering violations and complaints by owner. Otherwise all the information is historical data by property rather than by owner. Without it, we could show other properties currently connected to that owner and a history of violations/complaints at the selected property, but anything beyond that requires the who owns-what-when part.

I can look into moving it to the owner tab so that its a secondary loading thing rather than necessary for the Summary page.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android.

ssuffian commented 2 years ago

I moved it into the owner tab so that the initial loading screen is much faster!