Skip to content

Conversation

IvanDanchev
Copy link
Contributor

@IvanDanchev IvanDanchev commented Aug 21, 2025

@IvanDanchev IvanDanchev requested review from a team as code owners August 21, 2025 10:14
@IvanDanchev IvanDanchev marked this pull request as draft August 21, 2025 10:14
@IvanDanchev IvanDanchev marked this pull request as ready for review August 26, 2025 05:11
@dimodi
Copy link
Contributor

dimodi commented Aug 26, 2025

Add the Chat to the "INTERACTIVITY AND UX" category on the home page.

Comment on lines +29 to +31
TextField="Content"
AuthorIdField="UserId"
TimestampField="SentAt"
Copy link
Contributor

Choose a reason for hiding this comment

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

Use @nameof()

};

Messages.Add(newMessage);
await InvokeAsync(StateHasChanged);
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this really necessary? The component updates even without it.

SentAt = DateTime.Now.AddMinutes(-3)
}
};
private string CurrentUserId = "user1";
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
private string CurrentUserId = "user1";
private readonly string CurrentUserId = "user1";


public class ChatMessage
{
public string Id { get; set; }
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
public string Id { get; set; }
public string Id { get; set; } = string.Empty;

public class ChatMessage
{
public string Id { get; set; }
public string Content { get; set; }
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
public string Content { get; set; }
public string Content { get; set; } = string.Empty;

{
public string Id { get; set; }
public string Content { get; set; }
public string UserId { get; set; }
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
public string UserId { get; set; }
public string UserId { get; set; } = string.Empty;

public string Content { get; set; }
public string UserId { get; set; }
public DateTime SentAt { get; set; }
public List<FileSelectFileInfo> Attachments { get; set; }
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
public List<FileSelectFileInfo> Attachments { get; set; }
public List<FileSelectFileInfo>? Attachments { get; set; }

Comment on lines 151 to 162
public class ChatMessage
{
public string Id { get; set; }

public string AuthorId { get; set; }

public string AuthorName { get; set; }

public string Content { get; set; }

public DateTime Timestamp { get; set; }
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
public class ChatMessage
{
public string Id { get; set; }
public string AuthorId { get; set; }
public string AuthorName { get; set; }
public string Content { get; set; }
public DateTime Timestamp { get; set; }
}
public class ChatMessage
{
public string Id { get; set; } = string.Empty;
public string AuthorId { get; set; } = string.Empty;
public string AuthorName { get; set; } = string.Empty;
public string Content { get; set; } = string.Empty;
public DateTime Timestamp { get; set; }
}

@dimodi dimodi merged commit 5bc71da into master Aug 26, 2025
@github-actions github-actions bot deleted the chat-docs branch August 26, 2025 07:41
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.

4 participants