Should the latter GetAttendeeAsync in SessionAttendeeCheckIn be GetSessionAsync instead?

[gojanpaolo]

The class SessionAttendeeCheckIn currently have two definitions of GetAttendeeAsync

graphql-workshop/code/complete/GraphQL/Attendees/SessionAttendeeCheckIn.cs

Lines 35 to 43 in 2e81711

	public Task<Attendee> GetAttendeeAsync(
	AttendeeByIdDataLoader attendeeById,
	CancellationToken cancellationToken) =>
	attendeeById.LoadAsync(AttendeeId, cancellationToken);
	public Task<Session> GetAttendeeAsync(
	SessionByIdDataLoader sessionById,
	CancellationToken cancellationToken) =>
	sessionById.LoadAsync(AttendeeId, cancellationToken);

But it seems to me that the latter should be GetSessionAsync instead. Or am I missing something?

That part is specifically covered in 7-subscriptions.
... # 2 in https://github.com/ChilliCream/graphql-workshop/blob/master/docs/7-subscriptions.md#add-onattendeecheckedin-subscription

I'll be happy to create a PR if it needs correction

Btw, thank you for creating this workshop. I'm learning graphql and I'm having a great learning experience so far.

[michaelstaib]

Yeas that is correct .... do you want to do a pull-request for this?

[gojanpaolo]

@michaelstaib sure 👍