search

openstax / os-webview

:books: Front-end web application for the OpenStax web site
28 stars 3 forks source link

Submit current URL as tracking_parameters in adoption/interest forms #2598

Closed RoyEJohnson closed 2 months ago

RoyEJohnson commented 2 months ago

DISCO-264

RoyEJohnson commented 2 months ago

With JS files, I don't need to distinguish between JS and JSX. Almost all of them include JSX, but I don't use that suffix. (Eventually they'll all be TS or TSX.)

Dantemss commented 2 months ago

I know it works without the extension being .jsx, it's just nitpicking because they use the jsx templating and wouldn't be valid .js files without that. But it makes sense not to change just a couple of files if there are dozens more in the repo.