V3 compatibility does not seem to work with lowercase JSX

[Cristiano Calcagno]

Example (built in v3 mode):

open ReactV3
let _ = <div name="" /> // The value domProps can't be found in ReactDOMRe

CC @mattdamon108

[woonki]

Can you share the deps? compiler and rescript-react?

[Cristiano Calcagno]

"rescript": "^10.1.0-rc.2"
"@rescript/react": "^0.11.0-rc.1"

[woonki]

I’ll check and let you know.

[woonki]

I think this error is caused by removing include ReactDOM.Props https://github.com/rescript-lang/rescript-react/pull/65/files which is still used by V3. I think I need to revert it include ReactDOM_V3.Props back to fix the build issue.
What do you think? @cknitt

[Christoph Knittel]

If it's only about this one type, then I'd prefer to just add

type domProps = ReactDOM_V3.domProps

to legacy/ReactDOMRe.res.

[woonki]

If it's only about this one type, then I'd prefer to just add

type domProps = ReactDOM_V3.domProps

to legacy/ReactDOMRe.res.

Good point. I’ll make a PR to fix it.
How about adding the old type props either?

type props = ReactDOM_V3.Props.props

[Christoph Knittel]

Not sure if it is required, but I guess it won't hurt.

So

type domProps = ReactDOM_V3.Props.domProps
type props = ReactDOM_V3.Props.props

or equivalently, as there is an include Props in ReactDOM_V3:

type domProps = ReactDOM_V3.domProps
type props = ReactDOM_V3.props

[woonki]

I've run some tests, I found the type alias won't work. Only include ReactDOM_V3.Props fixes the issue, because the type alias is not working @derving(abstract). That's why the error msg was complaining about the value of domProps instead of type domProps.

[Christoph Knittel]

Ah, right, sorry, had not noticed that in the error message.

[woonki]

@cristianoc Can you check whether the issue is gone?

[Cristiano Calcagno]

@cristianoc Can you check whether the issue is gone?

Works well, thanks.