[FEATURE]-Trigger GenericCommMonitor Poll Inclusively

[jonnyarndt]

Is your feature request related to a problem? Please describe.
The PepperDash.Essentials.Core GenericCommunicationMonitor polling mechanism does not start polling unless the GenericCommunicationMonitor client is of type iSocketStatus. A device with a com or udp communication method type does not trigger BeginPolling() within the GenericCommunicationMonitor subsequently not triggering the constructor item with signature object Action pollAction or pollString.

Describe the solution you'd like
Recommend removing BeginPolling() out of the if (!IsSocket) forcing BeginPolling() no matter if the client type is of iSocketStatus or not. This allows EPI devices to utilize the GenericCommunicationMonitor constructor object Action pollAction or pollString to implement polling, independent of the control method defined.

Describe alternatives you've considered
Change the BeginPolling() method accessor to include public override. However, I like the solution above better.

Additional context
None

[ndorin]

@jonnyarndt go ahead and remove the if(!IsSocket) conditional around the BeginPolling() call inside the Start() method.

While you're at it, I would also recommend inside the Start() method adding a line to deregister for the client.BytesReceived and client.TextReceived events before registering. Otherwise, calling Start() multiple times will cause multiple event registrations:

	public override void Start()
	{
            if (MonitorBytesReceived) 
            {
                     Client.BytesReceived -= Client_BytesReceived;
		    Client.BytesReceived += Client_BytesReceived;
          }
            else
            {
                   Client.TextReceived -= Client_TextReceived;
                   Client.TextReceived += Client_TextReceived;
            }

                BeginPolling();
            
	}