Calendar Skill issue with Meeting room booking v8 Release

[gagankohli2000]

What project is affected?

Calendar Skill - Meeting room booking solutions

What language is this in?

c#

What happens?

FindMeetingRoomDialog.cs has few issues

Issue 1
Where are you setting state.MeetingInfo.MeetingRoomName?
As this value is null which result is error in GetMeetingRoom()
var state = await Accessor.GetAsync(sc.Context);

            state.MeetingInfo.UnconfirmedMeetingRoom = !string.IsNullOrEmpty(state.MeetingInfo.MeetingRoomName) ?
                    await SearchService.GetMeetingRoomAsync(state.MeetingInfo.MeetingRoomName) :
                    await SearchService.GetMeetingRoomAsync(state.MeetingInfo.Building, state.MeetingInfo.FloorNumber.GetValueOrDefault());

Issue 2
Floor number is hard coded to meetingRooms[0].FloorNumber;

if (state.MeetingInfo.FloorNumber == null)
{
state.MeetingInfo.FloorNumber = meetingRooms[0].FloorNumber;
state.MeetingInfo.MeetingRoomName = meetingRooms[0].DisplayName;
foreach (var room in meetingRooms)
{
if (room.FloorNumber != state.MeetingInfo.FloorNumber)
{
state.MeetingInfo.FloorNumber = null;
break;
}
}
}

What are the steps to reproduce this issue?

What were you expecting to happen?

Can you share any logs, error output, etc.?

Any screenshots or additional context?

[xhr0804]

For the issue1, the MeetingRoomName will only be set at the beginning of a dialog. For example "book the meeting room bjw 64", the "bjw 64" will be recognized as MeetingRoomName.
For the issue2, the logic is to check whether there is only one FloorNumber. If there is only one floor, we just use that floorNumber directly, or we will set it to Null and prompt to user.

[bobokids]

Close the issue for now. @gagankohli2000, if you have more questions/suggestions, please re-activate the issue.