search

MushroomObserver / mushroom-observer

A website for sharing observations of mushrooms.
https://mushroomobserver.org
MIT License
77 stars 25 forks source link

Disable link prefetching sitewide #2174

Closed nimmolo closed 3 months ago

nimmolo commented 3 months ago

Turbo (as of version 8, aka turbo-rails v2.0) defaults to prefetching the entire HTML of any page whose link you hover over.

This creates a ton of extra traffic and noise in the logs, and has unintended side-effects in our code, because of the way we store the last-visited observation for a user.

This PR disables link-prefetching site-wide.

It can alternatively be disabled on an element and its descendants with data-turbo-prefetch="false" - for example the navbar, or any link to an obs, but for now I believe we don't need it anywhere.

coveralls commented 3 months ago

Coverage Status

coverage: 94.454%. remained the same when pulling 9eedef337027593964d60d7da3852ea5f0494775 on disable-link-prefetching into 8f8dec86b70d7f602e119c87b1bed26f49cd84f0 on main.