Skip to content

Conversation

@hbollon
Copy link
Member

@hbollon hbollon commented Aug 27, 2025

No description provided.

@hbollon hbollon marked this pull request as draft August 27, 2025 09:54
@hbollon hbollon force-pushed the PB-1836-update-to-psycopg3 branch 4 times, most recently from a80cb8f to f873914 Compare August 27, 2025 12:21
@hbollon hbollon marked this pull request as ready for review August 27, 2025 13:07
@hbollon hbollon marked this pull request as draft August 28, 2025 08:22
@hbollon hbollon force-pushed the PB-1836-update-to-psycopg3 branch from f873914 to af60e24 Compare August 28, 2025 08:43
@msom
Copy link
Contributor

msom commented Sep 16, 2025

@hbollon I think you deleted the raise in get_db_connection (I guess raising a ConnectionError in case of tests are expected by flask and probably leads to some mocking when using test_client).

@hbollon
Copy link
Member Author

hbollon commented Sep 16, 2025

@msom Nice catch! I didn't pay attention to that.
It seems to solve my issues, thank you. I will update the PR

@hbollon hbollon force-pushed the PB-1836-update-to-psycopg3 branch from af60e24 to 56eff33 Compare September 18, 2025 12:07
@hbollon hbollon force-pushed the PB-1836-update-to-psycopg3 branch from 56eff33 to 8100f57 Compare September 18, 2025 12:18
@hbollon hbollon marked this pull request as ready for review September 18, 2025 12:30
@hbollon hbollon requested a review from msom September 18, 2025 12:30
@hbollon hbollon merged commit 4980245 into develop Sep 18, 2025
3 checks passed
@hbollon hbollon deleted the PB-1836-update-to-psycopg3 branch September 18, 2025 12:55
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.

3 participants