openerp-community-reviewer team mailing list archive
-
openerp-community-reviewer team
-
Mailing list archive
-
Message #05158
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