search

EnterpriseDB / docs

EDB Docs
https://www.enterprisedb.com/docs/
Apache License 2.0
44 stars 217 forks source link

Fix render loop on advanced search #5830

Closed josh-heyer closed 2 days ago

josh-heyer commented 2 days ago

What Changed?

Root cause of this was passing a string to <Configure> where it expected a number. The irritating bit about that was that it worked, it just kept re-rendering as the internal state was never an exact match with what was passed.

Took this opportunity to clean up a few other places that were triggering unnecessary re-rendering; page should be noticeably faster now.

Finally, re-enable analytics (which I'd disabled the other dayas a temporary mitigation; didn't fix the root issue, but did stop it from quickly crashing a browser by filling logs with errors)

github-actions[bot] commented 2 days ago

🚀 Deployed on https://deploy-preview-5830--edb-docs-staging.netlify.app

djw-m commented 2 days ago

Checked out on preview deploy, moving into production.