← Back to team overview

openerp-community-reviewer team mailing list archive

Re: [Merge] lp:~agilebg/server-env-tools/add_textual_database_name into lp:server-env-tools

 

Review: Needs Information code review and test

Hi, 

It works. But if the user tip an incorrect database name, there is a non managed error server-side.
OperationalError: FATAL:  database "test_module" does not exist

If the user has to input manually the database name my opinion is that the message has to be more user-friendly.(like when credentials are incorrect).

What do you think ?

Regards.
-- 
https://code.launchpad.net/~agilebg/server-env-tools/add_textual_database_name/+merge/209184
Your team Server Environment And Tools Core Editors is requested to review the proposed merge of lp:~agilebg/server-env-tools/add_textual_database_name into lp:server-env-tools.


References