Allow the Emulator user to specify a User ID

[Aliandi]

Fixes #1344

Description

This PR allows the Emulator users to specify a User ID which will be used within conversations.

Changes made

Functionality changes

To implement this feature we made the following changes:

• Add an input to the Emulator Settings so that the user can configure a GUID
• Change the startNewConversation method to choose between a randomly generated UserId, the UserId set in the props or the UserId set by the user.'
• When the Use custom id checkbox is set to false, the input is disabled and cleaned. When true, it is a required field and it gets enabled.
• The User ID field has a validation so that the user is not able to enter anything other than a GUID.
• Restarting a conversation with the same userId will choose between the one set by the user or the one set by default (if the first one is empty).
• Restarting a conversation with a new userId will generate a random ID the same way it was doing it until now.
• Remove Restart Conversation Split Button
• Add two new buttons Restart with Same User Id and Restart with New User Id to replace Restart Conversation Split Button.

UI Changes

Before

After

Testing

Manual testing

Automated testing
Added automated tests for the new settings options.

[coveralls]

Pull Request Test Coverage Report for Build 2369

• 10 of 13 (76.92%) changed or added relevant lines in 2 files are covered.
• 4 unchanged lines in 1 file lost coverage.
• Overall coverage increased (+0.03%) to 55.447%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
packages/app/client/src/ui/editor/emulator/emulator.tsx	8	11	72.73%

Files with Coverage Reduction	New Missed Lines	%
packages/app/client/src/ui/editor/emulator/emulator.tsx	4	75.3%

Totals
Change from base Build 2367:	0.03%
Covered Lines:	4898
Relevant Lines:	8326

---

💛 - Coveralls

[tonyanziano]

Nice work! I'll approve once tests are committed.

One thing I noticed that was when the checkbox is checked, and the GUID text field becomes active, the label styling still appears to be inactive:

I think that the color of the label text should match the color of the other active inputs.

[tonyanziano]

Great job! 🎉

[denscollo]

Great job! 🎉

Thanks! 🎉

[cwhitten]

Hi @Aliandi and @denscollo:

Justin's in-flight work in #1400 had problems reconciling with some of the changes made in this diff, so we reverted it in the hopes that when his branch lands @Aliandi can reissue the pull request against master again.

[cwhitten]

@denscollo @Aliandi this is ready to be re-opened against master. There has been a refactor to some of the code-paths involved so some changes will be necessary.