Skip to content

Documentation(872737)PostgreSQL DataBinding #4078

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
Open
wants to merge 2 commits into
base: hotfix/hotfix-v24.2.3
Choose a base branch
from
Open

Documentation(872737)PostgreSQL DataBinding #4078

wants to merge 2 commits into from

Conversation

Naveen-Palanivel
Copy link
Contributor

Documentation(872737)PostgreSQL DataBinding

@Naveen-Palanivel Naveen-Palanivel added the documentation Improvements or additions to documentation label Mar 14, 2024
@Naveen-Palanivel Naveen-Palanivel self-assigned this Mar 14, 2024
@SyncfusionBuild
Copy link
Contributor

Build Status: INQUEUE 🕒
Build Location: Documentation(872737)PostgreSQL-DataBinding

@SyncfusionBuild
Copy link
Contributor

Build Status: INPROGRESS 🔃
Build Location: Documentation(872737)PostgreSQL-DataBinding

@SyncfusionBuild
Copy link
Contributor

CI Status: FAILURE ❌
Source Compilation: SUCCESS ✅
Failure Reason: CI Failure with violation
Build Location: Documentation(872737)PostgreSQL-DataBinding
Image Alt Text Error(s): 0
Image Size Error(s): 0
Image Name Error(s): 0
FT Layout Syntax Error(s): 0
Front matter Error(s): 1
File Path Error(s): 0
CodeSnippet File Path Missing Error(s): 0
Toc Anchor Tags Validation Error(s): 0

@SyncfusionBuild
Copy link
Contributor

Build Status: INQUEUE 🕒
Build Location: Documentation(872737)PostgreSQL-DataBinding

@SyncfusionBuild
Copy link
Contributor

Build Status: INPROGRESS 🔃
Build Location: Documentation(872737)PostgreSQL-DataBinding

@SyncfusionBuild
Copy link
Contributor

CI Status: SUCCESS ✅
Source Compilation: SUCCESS ✅
Build Location: Documentation(872737)PostgreSQL-DataBinding
Image Alt Text Error(s): 0
Image Size Error(s): 0
Image Name Error(s): 0
FT Layout Syntax Error(s): 0
Front matter Error(s): 0
File Path Error(s): 0
CodeSnippet File Path Missing Error(s): 0
Toc Anchor Tags Validation Error(s): 0

MaithiliyK
MaithiliyK reviewed Mar 18, 2024
View reviewed changes
blazor/datagrid/connecting-to-database/postgresql-server.md
string ConnectionString = @"<Enter a valid connection string>";
string QueryStr = "SELECT * FROM public.\"Orders\" ORDER BY \"OrderID\"";
NpgsqlConnection sqlConnection = new(ConnectionString);
sqlConnection.Open();
Copy link
Collaborator

Choose a reason for hiding this comment

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

variable name must be in proper casing and meaningfull name

MaithiliyK
MaithiliyK reviewed Mar 18, 2024
View reviewed changes
blazor/datagrid/connecting-to-database/postgresql-server.md
// Create Npgsql Connection
NpgsqlConnection sqlConnection = new(ConnectionString);
sqlConnection.Open();
// Using NpgsqlDataAdapter and Query create connection with the database
Copy link
Collaborator

Choose a reason for hiding this comment

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

comments are duplicate

MaithiliyK
MaithiliyK approved these changes Mar 19, 2024
View reviewed changes
Copy link
Collaborator

@MaithiliyK MaithiliyK left a comment

Choose a reason for hiding this comment

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

changes are fine add the github sample link and merge this PR. Is the fig images are added in the topic ensured that

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MaithiliyK MaithiliyK MaithiliyK approved these changes

@VigneshNatarajan27 VigneshNatarajan27 Awaiting requested review from VigneshNatarajan27

Assignees

@Naveen-Palanivel Naveen-Palanivel

Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Naveen-Palanivel @SyncfusionBuild @MaithiliyK