feat: useData()

[lourot]

1. adding useData() to vike-react
2. while staying backward compatible with onBeforeRender-based projects
3. copying basic example to examples/onBeforeRender
4. changing basic example from using onBeforeRender to using data+useData()

[brillout]

👌 Good idea to use data.title.

WDYT of 6a87d62? I think we could even do it now and release a breaking change. Making our a life a little easier for replicating this PR to vike-vue and vike-solid. Thoughts?

[lourot]

Good idea, but I suggest we make it in several PRs:

1. we land this and release as [email protected]
2. I create a PR for bati to switch to data() hooks with useData() (Bati relies on onBeforeRender()+pageProps) (FYI @magne4000)
3. I create a follow-up PR removing support for onBeforeRender()+pageProps and we release it as [email protected]

WDYT?

[brillout]

👍 Ok good idea

…

On Sat 23. Dec 2023 at 23:14, Aurélien Lourot ***@***.***> wrote: Good idea, but I suggest we make it in several PRs: 1. we land this and release as ***@***.*** 2. I create a PR for bati to switch to data() hooks with useData() (Bati relies on onBeforeRender()+pageProps) (FYI @magne4000 <https://github.com/magne4000>) 3. I create a follow-up PR removing support for onBeforeRender()+ pageProps and we release it as ***@***.*** WDYT? — Reply to this email directly, view it on GitHub <#50 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAHVQRURN4AJH4QHY2IT2WTYK5JU5AVCNFSM6AAAAABBBAZV32VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRYGM3TSMRVGE> . You are receiving this because you commented.Message ID: ***@***.***>

[lourot]

Let me know if you need more time to review this or if this can land. Thanks :)

[brillout]

👍 Let me have a closer look.

[brillout]

LGTM :)