-
Notifications
You must be signed in to change notification settings - Fork 42
Add MongoDB Entity Framework Core integration documentation #323
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
base: release/13.2
Are you sure you want to change the base?
Conversation
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>
| </Pivot> | ||
|
|
||
| </Pivot> |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
| <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/" | ||
| /> |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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..."
| var mongo = builder.AddMongoDB("mongo") | ||
| .WithLifetime(ContainerLifetime.Persistent); |
There was a problem hiding this comment.
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.)
There was a problem hiding this comment.
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>
Plan: Add EF Core MongoDB Integration Documentation
mongodb-efcore-client.mdx)mongodb-efcore-get-started.mdx)Original prompt
💡 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.