----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120294/#review67083 -----------------------------------------------------------
Please improve a few minor issues. kexi/main/startup/KexiStartupFileHandler.cpp <https://git.reviewboard.kde.org/r/120294/#comment46817> We shouldn't leave debug code, should we? kexi/migration/importwizard.cpp <https://git.reviewboard.kde.org/r/120294/#comment46823> We don't need dstServerTypeCombo any longer. kexi/migration/importwizard.cpp <https://git.reviewboard.kde.org/r/120294/#comment46821> Please change: 'pasword' -> 'password' kexi/migration/importwizard.cpp <https://git.reviewboard.kde.org/r/120294/#comment46822> More meaningful name than "ok" should be better :) kexi/migration/importwizard.cpp <https://git.reviewboard.kde.org/r/120294/#comment46824> Should we leave unused commented code? If not, please remove. - Michal Poteralski On Sept. 20, 2014, 12:06 a.m., Jarosław Staniek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/120294/ > ----------------------------------------------------------- > > (Updated Sept. 20, 2014, 12:06 a.m.) > > > Review request for Calligra, Andrius da Costa Ribas, Oleg Kukharchuk, Adam > Pigg, Dimitrios Tanis, Tomasz Olszak, Radosław Wicik, and Wojciech Kosowicz. > > > Bugs: 336556, 336557 and 336558 > http://bugs.kde.org/show_bug.cgi?id=336556 > http://bugs.kde.org/show_bug.cgi?id=336557 > http://bugs.kde.org/show_bug.cgi?id=336558 > > > Repository: calligra > > > Description > ------- > > Many fixes for Kexi database import assistant > > * Import Database" assistant may require password to access > source or destination server but it does not ask. Then connection > error appears. > * reload source database list if user pressed back and selected different > connection > * improve saving recent directories > * do not ask twice about destination filename > * in addition to title, properly ask for database name for server destination > databases > * properly open imported database (file) > > > Diffs > ----- > > kexi/widget/KexiDBTitlePage.ui 14b811ee8e06733df1bfdd14829befaec12e000e > kexi/main/startup/KexiNewProjectAssistant.cpp > c016cfc2c56f5cb7d6eeb2ebdfe71b3dff8969f6 > kexi/main/startup/KexiStartupFileHandler.h > ee0ecc0c87c499a1753c803098621201050c4f97 > kexi/main/startup/KexiStartupFileHandler.cpp > 2eb6b21534b3bac78792dce8c4e08c3b47cc2508 > kexi/migration/CMakeLists.txt b84ad51694f627870fff2d69ec3f71427982f87b > kexi/migration/importwizard.h ce328168d9643115aade796647689dd1159331fe > kexi/migration/importwizard.cpp 7cbea8165ba97ecce67c77b4d25dca1c0a088485 > > Diff: https://git.reviewboard.kde.org/r/120294/diff/ > > > Testing > ------- > > tested all combinations of server/file import > > > Thanks, > > Jarosław Staniek > >
_______________________________________________ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel