Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 30, 2026

Plan: Add EF Core MongoDB Integration Documentation

  • Create MongoDB EF Core client integration documentation file (mongodb-efcore-client.mdx)
  • Create MongoDB EF Core get started guide (mongodb-efcore-get-started.mdx)
  • Update MongoDB get started guide to include EF Core option
  • Update EF Core overview to include MongoDB option
  • Add MongoDB EF Core to sidebar navigation
  • Address code review feedback
    • Fix nested Pivot key attributes
    • Fix code indentation to use standard spacing
    • Shorten card descriptions
  • Review and finalize
Original prompt

It looks like we need add details for the EF Core MongoDB client integration, that's been added as part of 13.2 - here's the source of truth: https://github.com/dotnet/aspire/tree/main/src/Components/Aspire.MongoDB.EntityFrameworkCore


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 3 commits January 30, 2026 16:33
Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com>
Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com>
Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com>
Copilot AI changed the title [WIP] Add details for EF Core MongoDB client integration Add MongoDB Entity Framework Core integration documentation Jan 30, 2026
Copilot AI requested a review from IEvangelist January 30, 2026 16:42
Comment on lines +169 to +171
</Pivot>

</Pivot>
Copy link
Member

Choose a reason for hiding this comment

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

@copilot I think this is wrong, please double check your work here... you have double closing pivots, in multiple places.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed in commit f1280df. Added key="mongo-client" attribute to all nested Pivot components so they properly reference the parent PivotSelector.

Comment on lines 110 to 125
<CardGrid>
<LinkCard
title="Understand the MongoDB hosting integration"
description="Discover how to configure persistence, MongoDB Express resources, and more."
href="/integrations/databases/mongodb/mongodb-host/"
/>
<LinkCard
title="Understand the MongoDB EF Core client integration"
description="Discover how to enrich DbContext, configure settings, and more."
href="/integrations/databases/mongodb/mongodb-efcore-client/"
/>
<LinkCard
title="Understand the MongoDB client integration"
description="Learn about the MongoDB Driver client integration."
href="/integrations/databases/mongodb/mongodb-client/"
/>
Copy link
Member

Choose a reason for hiding this comment

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

The text might get too long here on these cards, please take screenshots and use best judgement. Ideally, the right chevron won't be squished.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed in commit f1280df. Shortened descriptions:

  • "Discover how to configure persistence..." → "Configure persistence..."
  • "Discover how to enrich DbContext..." → "Enrich DbContext..."

Comment on lines 41 to 42
var mongo = builder.AddMongoDB("mongo")
.WithLifetime(ContainerLifetime.Persistent);
Copy link
Member

Choose a reason for hiding this comment

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

We've had feedback that this style of formatting is not ideal. Please try to stylize C# code such that fluent APIs on a newline are simply indented a standard amount (not aligned with the line above .Call.)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed in commit f1280df. Updated code formatting to use standard 4-space indentation instead of alignment.

Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com>
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.

2 participants