Hi Federico, On Fri, Aug 6, 2010 at 6:31 PM, Federico Paparoni <federico.papar...@gmail.com> wrote: > 2010/8/6 Bertrand Delacretaz <bdelacre...@apache.org> >> ...-> suggest a README.txt at the top of the davidgsoc2010 code tree that >> describes the installation. >> > > Yes, it will be also described in the documentation....
Ok, please avoid duplicating the info - best is probably to have the basic install/start information in README (where it's easy to quickly fix if something changes, and stays in sync with the code), and refer to that in the docs, without duplicating details. >> ...Sometimes saving an entry doesn't work, server says >> >> Error while processing /content/david/2010/08/07/about_sling >> javax.jcr.AccessDeniedException: Access denied >> >> even though I could save another entry seconds before.... >> > I don't understand why. There is the check to see if you are logged and I > never received this error. > Can you tell me the exact steps you performed? Which browser are you using? Creating and saving entries a few times led to this problem, but it seems semi-random. For now I think you could just improve the client-side error reporting, i.e. display the server's error messages or at least http status code instead of just [object xmlhttprequest] in the client's error message. >>... The libraries bundle should be split in two: ckeditor and swfobject >> for tags display. Or more than two if it contains other independent >> components - the idea is to have one bundle per independent component. >> The convention is to use /libs, not /libraries for those types of >> things. >> >> -> split bundle >> > > I can split the libraries into two or more bundles, but I think it isn't > useful. > It can be useful if there is a mechanism to change these components with > others (other layout, other JS framework etc. etc.). But to do that I had to > focus more on the project that on Sling.... It is not useful in the context of your single project, but if someone would like to reuse the ckeditor stuff, as is now they get extra stuff in the bundle that they don't need. > Anyway if you think it's a "must-have" feature (also for a future > development) I can separate them.... It's not a must have at this point, better concentrate on the priorities outlined in my "davidgsoc2010, suggested next steps" email of last Friday here. -Bertrand
