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 the ability to customize JDBC Driver/Datasource defaults if not using standard options #1092

Merged
merged 1 commit into from
May 9, 2017

Conversation

mtho11
Copy link
Contributor

@mtho11 mtho11 commented Apr 20, 2017

Add the capability to provide an "Other..." option to customize both JDBC Driver defaults and Add Datasource (they both use the same angular service data). If one is using something other than the standard driver defaults we have provided, then give the user blank fields to add their own settings without having to delete each fields defaults. This is really just a usability feature for non-standard drivers.

other

Here is a screenshot further illustrating the 'Other...' options blanking out the standard defaults provided:

blank-options

@mtho11
Copy link
Contributor Author

mtho11 commented Apr 20, 2017

BTW: last commit here is the only commit relevant to this PR: 756280a since previous commit is required for this.

@mtho11
Copy link
Contributor Author

mtho11 commented Apr 20, 2017

@miq-bot add_label middleware, enhancement, angular dialogs

@mtho11
Copy link
Contributor Author

mtho11 commented Apr 20, 2017

@miq-bot assign @abonas

@abonas
Copy link
Member

abonas commented Apr 20, 2017

you can rebase since the prerequisite was merged :)
@miq-bot add_label enhancement
@Loicavenel @jdoyleoss this is an enhancement, but a small one, how about adding it to fine?

@@ -32,6 +32,9 @@ function MwAddDatasourceService($http, $q) {
{id: 'MYSQL', label: 'MySql', name: 'MySqlDS', jndiName: 'java:jboss/datasources/MySqlDS',
driverName: 'mysql', driverModuleName: 'com.mysql', driverClass: 'com.mysql.jdbc.Driver',
connectionUrl: '://localhost:3306/db_name'},
{id: 'OTHER', label: 'Other...', name: '', jndiName: '',
Copy link
Member

Choose a reason for hiding this comment

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

unlike the other driver names that are more vendor specific, "Other" should probably be translation ready and not hardcoded

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yep, good catch.

@miq-bot
Copy link
Member

miq-bot commented Apr 20, 2017

Checked commit https://github.com/mtho11/manageiq-ui-classic/commit/e45db5c961cf1e92d07421d7b9375649c22ef639 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
0 files checked, 0 offenses detected
Everything looks good. 🍰

@abonas
Copy link
Member

abonas commented Apr 24, 2017

@miq-bot assign @martinpovolny
@Loicavenel @jdoyleoss this is an enhancement, but a small one, how about adding it to fine?

@miq-bot miq-bot assigned martinpovolny and unassigned abonas Apr 24, 2017
@abonas
Copy link
Member

abonas commented May 4, 2017

given that no response was received regarding fine
@miq-bot add_label fine/no
@martinpovolny merge please?

@miq-bot miq-bot added the fine/no label May 4, 2017
@bronaghs
Copy link

bronaghs commented May 4, 2017

@martinpovolny - bump

@martinpovolny martinpovolny merged commit 0ef91e1 into ManageIQ:master May 9, 2017
@martinpovolny martinpovolny added this to the Sprint 61 Ending May 22, 2017 milestone May 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants