fix(react): enable event names with forward slash

[alexprudhomme]

fix #473

Pull request checklist

Please check if your PR fulfills the following requirements:

• [x] Tests for the changes have been added (for bug fixes / features)
• [x] Docs have been reviewed and added / updated if needed (for bug fixes / features)
• [x] Build (npm run build) was run locally for affected output targets
• [x] Tests (npm test) were run locally and passed
• [x] Prettier (npm run prettier) was run locally and passed

Pull request type

Please check the type of change your PR introduces:

• [ ] Bugfix
• [x] Feature
• [ ] Code style update (formatting, renaming)
• [ ] Refactoring (no functional changes, no api changes)
• [ ] Build related changes
• [ ] Documentation content changes
• [ ] Other (please describe):

What is the current behavior?

Issue URL:

What is the new behavior?

• enables event names with forward slash in react-output-target

Does this introduce a breaking change?

• [ ] Yes
• [x] No

Other information

[alexprudhomme]

@alexprudhomme I tested this out with your / event naming, but doesn't seem to work with that convention. As far as I know, React requires strict camel-casing for event handlers, so if you wanted to have event names with a / in them, you would need to update the logic here to also perform those replacements

Hey, @tanner-reits thanks for the useful tip. I changed the implementation and added a test. Let me know if you need anything more.

[alexprudhomme]

@tanner-reits Hi, Will you be able to make a release soon with this fix ? Last one was on august 28th. Thanks !

[christian-bromann]

@alexprudhomme I am working on it as we speak.