[fixed] Allow empty classNames for body

[jitinmaher]

Fixes #[664] and #[721].

#[664]: This PR allows empty classNames for body and portal when bodyOpenClassName is set with no value( null or "" )

#[721]: The PR also regenerates package-lock.json which fixes event-stream-3.3.6 issue caused because of the recent security incident

Acceptance Checklist:

• The commit message follows the guidelines in CONTRIBUTING.md.
• Documentation (README.md) and examples have been updated as needed.
• If this is a code change, a spec testing the functionality has been added.
• If the commit message has [changed] or [removed], there is an upgrade path above.

[jitinmaher]

@diasbruno Not sure why build is failing. Following is the error from travis ci:

npm ERR! code E404
npm ERR! 404 Not Found: [email protected]

Could this be related to the event-stream incident ?

Issue reported here

[diasbruno]

@jitinmaher Thanks for working on this.
Yep, that seems to be the case.
The only thing missing is to update the documentation and, once we have the tests passing, we can merge this PR.

[jitinmaher]

Thanks for the review @diasbruno . I have updated the documentation.

[diasbruno]

Good job. I believe the only page left to update is the docs/README.md.

[jitinmaher]

@diasbruno docs/README.md has been Updated 👍

[jitinmaher]

@diasbruno Updated the lock file to fix the event-stream issue. Builds and test cases are passing 👍.

[diasbruno]

Good job, @jitinmaher! I'll finish the review later and merge this. Probably, I'll release a new version tomorrow.

[jitinmaher]

Thanks @diasbruno . Looking forward for the new version!!

[jitinmaher]

Hey @diasbruno , any updates on the merge?

[coveralls]

Coverage remained the same at 86.56% when pulling f5dd570 on jitinmaher:master into 8d8f476 on reactjs:master.

[jitinmaher]

Ping @diasbruno

Sorry to bother you again. Any update on the merge?

[diasbruno]

No problem, @jitinmaher...actually, thanks for remind me to do it.

[diasbruno]

@jitinmaher thank you for having a look on this issue and for your patience to merge this. Good job!

[jitinmaher]

@diasbruno Thank you so much for the merge 👍

[diasbruno]

Released version 3.7.1.

[jitinmaher]

Thanks for the version update @diasbruno 🙂