Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NServiceBus step by step tutorial cleanup #6903

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

andreasohlund
Copy link
Member

@andreasohlund andreasohlund commented Nov 8, 2024

As part of preparing to run the 2-day NSB course, I went through the tutorials and found some things to clean up.

All solutions have been smoke-tested

I will make corresponding changes to https://github.com/Particular/Workshop.NServiceBus

Related cleanup of the saga tutorial #6898

@andreasohlund andreasohlund marked this pull request as ready for review November 10, 2024 14:11
}

#region InputLoopService

public class InputLoopService(IMessageSession messageSession, ILogger<InputLoopService> logger) : BackgroundService
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since we are interacting with the console anyway I figured that it would be more consistent to continue to use Console.WriteLine rather than mixing with logging

## Handling events

Create a handler class by implementing `IHandleMessages<T>` where `T` is the type of the event message.

snippet: EventHandler

> [!NOTE]
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since all transports we currently support (since msmq is not out of support) supports native pubsub I felt that this part was not relevant for a tutorial

@@ -20,7 +20,7 @@ In this first lesson, which should take 10-15 minutes, you will create your firs

Let's build something simple to give NServiceBus a try.

This tutorial uses NServiceBus version 8, .NET 6, and assumes an up-to-date installation of [Visual Studio 2022](https://www.visualstudio.com/downloads/).
This tutorial uses NServiceBus version 9, .NET 8, and assumes an up-to-date installation of [Visual Studio 2022](https://www.visualstudio.com/downloads/).

> [!NOTE]
Copy link
Member Author

@andreasohlund andreasohlund Nov 10, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm unsure if this note makes sense since this is targeted at users learning to use our latest version.

@andreasohlund
Copy link
Member Author

@nix-bohon FYI since I know you are going through the tutorials

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant