Skip to content

Correct PostreSQL connection string & Allow targets to customize default database name in connection examples#1577

Merged
Eugeny merged 11 commits intowarp-tech:mainfrom
SteezyCougar:customize-connection-string
Jan 24, 2026
Merged

Correct PostreSQL connection string & Allow targets to customize default database name in connection examples#1577
Eugeny merged 11 commits intowarp-tech:mainfrom
SteezyCougar:customize-connection-string

Conversation

@SteezyCougar
Copy link
Copy Markdown
Contributor

Adds support for targets to specify a custom default database name that will be used in connection string examples shown to users.

Changes

  • Adds default_database_name field to target configuration (MySQL and Postgres targets only)
  • Updates connection instruction examples to use the custom database name when specified
  • Falls back to "database-name" placeholder if not set
  • Adds database migration to add the new column

UI Changes

  • Adds input field in target configuration for setting default database name
  • Field is only visible for MySQL and Postgres targets
  • Connection instructions modal updates reactively when the value changes

Bug Fix

  • Fixes PostgreSQL connection command example to use -W flag instead of invalid --password flag

@SteezyCougar
Copy link
Copy Markdown
Contributor Author

Now example strings look like:

psql -U '[email protected]#STEEZY-DATABASE' --host 'warp.steezycougar.com' --port 55432 -W my-custom-db-name-example

@SteezyCougar
Copy link
Copy Markdown
Contributor Author

image

@SteezyCougar SteezyCougar marked this pull request as draft December 1, 2025 16:20
@SteezyCougar
Copy link
Copy Markdown
Contributor Author

I'll revise this PR within the next several days most likely, but for right now it's still WIP

@SteezyCougar SteezyCougar force-pushed the customize-connection-string branch from cbea763 to 7b72336 Compare January 9, 2026 23:09
@SteezyCougar SteezyCougar force-pushed the customize-connection-string branch from 7b72336 to 48bfbc9 Compare January 9, 2026 23:17
@SteezyCougar SteezyCougar marked this pull request as ready for review January 22, 2026 05:59
@SteezyCougar
Copy link
Copy Markdown
Contributor Author

@Eugeny this should be ready for review again! Apologies on taking so long, had a few migration conflicts and then got distracted with some other personal projects

@Eugeny
Copy link
Copy Markdown
Member

Eugeny commented Jan 24, 2026

Thank you! I've moved defaultDatabaseName into TargetOptions though, where all target type specific options live

@Eugeny Eugeny merged commit 31598a4 into warp-tech:main Jan 24, 2026
14 checks passed
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