Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Remove connectionName from "Add Npgsql database context with enrichment" code example #2337

Closed
ncarandini opened this issue Dec 27, 2024 · 0 comments · Fixed by #2548
Closed

Remove connectionName from "Add Npgsql database context with enrichment" code example #2337

ncarandini opened this issue Dec 27, 2024 · 0 comments · Fixed by #2548
Assignees
@IEvangelist
Labels
doc-bug Problem with the content; needs to be fixed [org][type][category] in-pr okr-quality Content-health KR: Concerns article defects/freshness or build warnings. Pri1 High priority, do before Pri2 and Pri3 📌 seQUESTered Identifies that an issue has been imported into Quest.

Comments

@ncarandini
Copy link

ncarandini commented Dec 27, 2024 •
edited by sequestor bot
Loading

Type of issue

Code doesn't work

Description

In the Add Npgsql database context with enrichment code example, the line:

connectionName: "postgresdb",

is invalid because, as is stated few lines later, "The EnrichNpgsqlDbContext won't make use of the ConnectionStrings configuration section since it expects a DbContext to be registered at the point it's called.

Page URL

https://learn.microsoft.com/en-us/dotnet/aspire/database/postgresql-entity-framework-integration?tabs=dotnet-cli

Content source URL

https://github.com/dotnet/docs-aspire/blob/main/docs/database/postgresql-entity-framework-integration.md

Document Version Independent Id

b0430379-092c-6361-27aa-a4c48d6b641e

Article author

@IEvangelist

Associated WorkItem - 372041
@ncarandini ncarandini changed the title Remove connectionName: "postgresdb" from **Add Npgsql database context with enrichment** code example Remove connectionName: "postgresdb" from "Add Npgsql database context with enrichment" code example Dec 27, 2024
@ncarandini ncarandini changed the title Remove connectionName: "postgresdb" from "Add Npgsql database context with enrichment" code example Remove connectionName from "Add Npgsql database context with enrichment" code example Dec 27, 2024
@alistairmatthews alistairmatthews mentioned this issue Feb 6, 2025
Merged
@IEvangelist IEvangelist self-assigned this Feb 10, 2025
@IEvangelist IEvangelist added 🗺️ reQUEST Triggers an issue to be imported into Quest. Pri1 High priority, do before Pri2 and Pri3 doc-bug Problem with the content; needs to be fixed [org][type][category] okr-quality Content-health KR: Concerns article defects/freshness or build warnings. labels Feb 10, 2025
@dotnetrepoman dotnetrepoman bot added 🗺️ mapQUEST Only used as a way to mark an issue as updated. RepoMan should instantly remove it. and removed 🗺️ mapQUEST Only used as a way to mark an issue as updated. RepoMan should instantly remove it. labels Feb 10, 2025
@sequestor sequestor bot added 📌 seQUESTered Identifies that an issue has been imported into Quest. and removed 🗺️ reQUEST Triggers an issue to be imported into Quest. labels Feb 11, 2025
@dotnetrepoman dotnetrepoman bot added the 🗺️ mapQUEST Only used as a way to mark an issue as updated. RepoMan should instantly remove it. label Feb 13, 2025
@IEvangelist IEvangelist moved this from 🔖 Ready to 👀 In review in dotnet/docs February 2025 sprint project Feb 13, 2025
@dotnetrepoman dotnetrepoman bot removed the 🗺️ mapQUEST Only used as a way to mark an issue as updated. RepoMan should instantly remove it. label Feb 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@IEvangelist IEvangelist

Labels
doc-bug Problem with the content; needs to be fixed [org][type][category] in-pr okr-quality Content-health KR: Concerns article defects/freshness or build warnings. Pri1 High priority, do before Pri2 and Pri3 📌 seQUESTered Identifies that an issue has been imported into Quest.
Projects
dotnet/docs February 2025 sprint project
Status: 👀 In review
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixes to the PostgreSQL EF integration article alistairmatthews/docs-aspire
2 participants
@ncarandini @IEvangelist