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.

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

Fixed an issue where deleting a database from the properties tab of d… #8478

Closed
wants to merge 1 commit into from

Conversation

anilsahoo20
Copy link
Contributor

…atabase collections and immediately opening its properties resulted in a connection lost error. #6503

…atabase collections and immediately opening its properties resulted in a connection lost error. pgadmin-org#6503

# Extracting the other connections such as QT, ERD, or PSQL
# connections of current database
query_tool_connections = [
Copy link
Contributor

Choose a reason for hiding this comment

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

Connections could be external from other than pgadmin too. Like psql, JDBC connection etc. You should check all the conenctions on the database not only from the pgadmin.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Okay

Copy link
Contributor

@yogeshmahajan-1903 yogeshmahajan-1903 left a comment

Choose a reason for hiding this comment

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

Please check logic to get existing external connections too

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