Automatically add getId prop to dynamic routes

[nandorojo]

For dynamic paths like /artists/[id], we should add getId to the Screen:

const dynamicPath = 'id'

return <Screen getId={({ params }) => params?.[dynamicPath]} />

The getId function lets you push the same screen multiple times, if the ID differs (for example, pushing multiple artists on top of each other).

[EvanBacon]

Good idea, this has been added in expo-router@0.0.33

[nandorojo]

Related: https://twitter.com/satya164/status/1567162295767207938?s=20&t=wzLSKI6opRrOjKGahJJw_w

navigate no longer goes back to a specific screen. Rather, you need to use popTo for that, which is better IMO. react-navigation/react-navigation#10818

[nandorojo]

It appears that getId is actually preserving the previous behavior, which is not exactly desired. Previously I used getId so that we could push multiple Artist screens on top of each other. However, given that it now is no longer needed to keep pushing the same screen over and over, we may no longer need getId here.

So once expo-router upgrades to React Navigation v7, I think we can remove this option.