LSE

Results 30 comments of LSE

Note: I didn't remove the `type="submit"` on the button as the JS code depends on it, see: https://github.com/odoo/odoo/blob/5771f6c3177d4844d8a981378ddc8825e54280a9/addons/website_event/static/src/js/website_event.js#L43 If I modify the JS code at this place it would have...

Several things that I would like to be reviewed on (as I'm not used to do documentation PR): - Should I remove the part in which I "advertise" a version...

@qle-odoo can I have your opinion as the dev on the features mentioned ?

@mivu-odoo could you please review it ?

@mivu-odoo Thank you for your comment and review. Note that I did needed to fix some of your suggestions (generally because the indentation was wrong so it was triggering warning...

@mivu-odoo commits squashed! Should I contact someone in particular to merge it in the documentation ?

@AntoineVDV done as you asked! I did indeed forgot about pngquant (even if i mentioned on the matter previously). is there something that I need to change ?

@aab-odoo (or anyone who review this PR ) Hey :) Would you mind reviewing this PR please? I'm not a big fan of the changes that I proposed as I...

@tranlam20220101 I don't really understand the point that you are trying to make with your screenshot, could you give me more details on it please ?