hermes landing page

[bruno-garcia]

After hermes support was added, a few users had issues.

So far all cases I noticed they were using old SDK or on prem Sentry.

Anything else we need to point out here?

[HazAT]

Also, one major thing that's wrong here
Everything that lives in /clients/ are the old outdated docs
/platforms/ are the new so this has to go there.

[bruno-garcia]

I have a call, after that I'll move it to the right place.
For now please let's iron out the content. It'll go under /plat

[marandaneto]

They have good docs about how to enable it and also to check the min. requirements, so I'd point them out to the official docs.
Something like:

Hermes is disabled by default, if you want to enable it and check its minimum requirements, please have a look at https://reactnative.dev/docs/hermes

[bruno-garcia]

@marandaneto @HazAT @Swatinem @MimiDumpling
What's missing to be able to merge this?

Besides I'm sure a snapshot file change I'll need to commit to make CI happy ;)

[MimiDumpling]

@bruno-garcia lemme do a quick spelling and language check. Other than that, it looks pretty good! Thanks for updating the docs 🌻

[Swatinem]

The sentry-cli version is also important.
Also, some customers that reported problems had customized the way they were building their code. In particular they were uploading the wrong source maps with sentry-cli.
Maybe those steps should be highlighted somewhere?

[MimiDumpling]

🌻 small suggestions. Thanks for updating Docs!

[bruno-garcia]

@Swatinem

Maybe those steps should be highlighted somewhere?

What steps would you suggest?

[marandaneto]

LGTM, if more questions come up that would not solve through this doc's, we can make it better later.

[bruno-garcia]

@HazAT I've moved to platforms already

[Swatinem]

What steps would you suggest?

Just to clarify which source maps should be uploaded, since hermes creates two intermediate sourcemaps, which should be avoided. And at least the outdated sentry-cli was using one of the wrong ones.