Skip to content

Ensure FirstName and LastName Follow Pascal Case in QuickGrid Component #60340

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

Merged
merged 1 commit into from
Feb 13, 2025

Conversation

shethaadit
Copy link
Contributor

PR Title:
Ensure FirstName and LastName Follow Pascal Case in QuickGrid Component

Description:
This PR updates the Person record in the QuickGrid component to ensure that firstName and lastName follow Pascal Case as FirstName and LastName. This change improves code consistency and aligns with standard C# naming conventions.

Changes:

  • Renamed firstName to FirstName and lastName to LastName in the Person record.
  • Updated all references in the QuickGrid component to use Pascal Case.
  • Ensured filtering and styling logic correctly references FirstName instead of firstName.

These changes enhance readability and maintainability while following best practices in .NET development.

Fixes #60325

@Copilot Copilot AI review requested due to automatic review settings February 12, 2025 18:44
@shethaadit shethaadit requested a review from a team as a code owner February 12, 2025 18:44
@ghost ghost added the area-blazor Includes: Blazor, Razor Components label Feb 12, 2025
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

@dotnet-policy-service dotnet-policy-service bot added the community-contribution Indicates that the PR has been added by a community member label Feb 12, 2025
Copy link
Member

@javiercn javiercn left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution

@javiercn javiercn merged commit 2edcf3b into dotnet:main Feb 13, 2025
27 checks passed
@dotnet-policy-service dotnet-policy-service bot added this to the 10.0-preview2 milestone Feb 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-blazor Includes: Blazor, Razor Components community-contribution Indicates that the PR has been added by a community member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Non-Pascal property casing in SampleQuickGridComponent Person record?
2 participants