search

BuddhaNexus / buddhanexus-frontend-next

BuddhaNexus Frontend 2.0
buddhanexus-frontend-next.vercel.app
2 stars 1 forks source link

Review `multi_lingual` param #117

Open aminahbl opened 6 months ago

aminahbl commented 6 months ago

The initial task is to define how this param should work. Update the ticket from there.

Previously on Slack:

Hubert one thing I wanted to call out is that IMO we should change how the multi_lingual param works. It currently calls the API on the client-side after the first render and this breaks SSG / pre-rendering the page shouldn’t re-render after being visited with filters applied @Sebastian Nehrdich I think if we change this param on the API side to include all languages by default (instead of none), we can get rid of this API call on the FE side (edited)

Also see https://github.com/BuddhaNexus/buddhanexus-frontend-next/pull/90#discussion_r1375272080 and the prefetchDbResultsPageData function in the code base.