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

Add jdbcUrl as configuration parameter #10

Merged
merged 1 commit into from
Sep 17, 2020
Merged

Conversation

TimoKramer
Copy link
Member

next.jdbc can take urls via :jdbcUrl to configure your connection.
For that to work it was necessary to make dbtype optional as
configuration parameter and add jdbcUrl as optional configuration
parameter. Now it is possible to pass the configuration partially
as url and as key-value-pairs to configure the connection.

@TimoKramer TimoKramer requested a review from whilo September 16, 2020 17:03
@TimoKramer TimoKramer self-assigned this Sep 16, 2020
@TimoKramer TimoKramer added the bug Something isn't working label Sep 16, 2020
@TimoKramer TimoKramer force-pushed the 8-jdbcUrl-missing branch 4 times, most recently from 6784587 to 4cb469c Compare September 17, 2020 07:04
next.jdbc can take urls via :jdbcUrl to configure your connection.
For that to work it was necessary to make dbtype optional as
configuration parameter and add jdbcUrl as optional configuration
parameter. Now it is possible to pass the configuration partially
as url and as key-value-pairs to configure the connection.

- Closes #8
- Updated README
- New CircleCI step for integrationtesting with postgresql and
  mariadb
@TimoKramer TimoKramer requested review from kordano and removed request for whilo September 17, 2020 08:22
@kordano kordano merged commit 27658d7 into development Sep 17, 2020
@TimoKramer TimoKramer deleted the 8-jdbcUrl-missing branch October 1, 2020 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants