[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Bug#674023: [RFR] templates://ledgersmb/{templates}



On Fri, 2012-05-25 at 01:10 -0400, Christian PERRIER wrote:
> Please find, for review, the debconf templates and packages descriptions
>  for the ledgersmb source package.
> 

> I'm not entirely convinced that we should mention "setup.pl". Better
> talk about the "Configure database" part of the web interface or
> something like this.

   But that's how it's referred to, as "setup.pl".  


> 
> I also tried to avoid the "for for" construction....Justin will
> probably have better suggestions.

   Yes, that is a bit awkward.

> 
> BTW, don't you use a password confirmation template et something to
> compare the two passwords entered? That requires some coding in the
> config scripts but makes things more robust.
> 
> If you want to do so, I have ready standard debconf templates for
> this..:-)
> (password confirmation template and "passwords do not match" error template)

    I'm still unfamiliar with using debconf and so would certainly be
interested in taking a look at that.


> -Description: Financial accounting and ERP program
> +Description: financial accounting and ERP program
>   LedgerSMB is a double-entry financial accounting and Enterprise Resource
> - Planning system which offers Accounts Receivable, Accounts Payable and General
> - Ledger tracking as well as inventory control and fixed assets handling. The
> + Planning system which offers "Accounts Receivable", "Accounts Payable" and "General
> + Ledger" tracking as well as inventory control and fixed assets handling. The

   It's unclear to me why those need to be in quotes like that?


> Very good package description!

   Well, Raphaël was rather insistent on the description being better
than what I originally started with.<g>






Jame



Reply to: