Particular / ServiceInsight

Advanced debugging for NServiceBus
http://particular.net/serviceinsight
Other
16 stars 32 forks source link

Sequence Diagram: Display hostname #558

Open cjoergensen opened 8 years ago

cjoergensen commented 8 years ago

Hi

Would you consider displaying the host in the Sequence Diagram?

Mock: seqmock

distantcam commented 8 years ago

Yes we can certainly consider that.

@sergioc Any thoughts on the proposed design above?

sergioc commented 8 years ago

How about simply appending @HOSTNAME to the endpoint name?

cjoergensen commented 8 years ago

@sergioc that could work...

It would be fancy if it worked like the "Show endpoints" feature in the flow digram, so you turn on/off as needed.

sergioc commented 8 years ago

It would be fancy if it worked like the "Show endpoints" feature in the flow digram, so you turn on/off as needed.

@cjoergensen sounds reasonable to me.

distantcam commented 8 years ago

How about simply appending @HOSTNAME to the endpoint name?

The endpoint name could be long, and we currently wrap it. I prefer the separation of showing the hostname in a new line rather than just appending it to the endpoint name.

sergioc commented 8 years ago

Here, with further de-emphasis of the host name:

screenshot 2016-03-16 13 40 03

@cjoergensen OK to show the host name below the endpoint name or is there a reason for it to better show above the endpoint name?

cjoergensen commented 8 years ago

@sergioc Below makes sense. Matches the syntax from the 'MessageEndpointMappings' ConfigurationSection