Raffael Meyer
Raffael Meyer
Tested on `develop`, can't reproduce any errors. As for removing draw controls, the following **Client Script** should work fine: ```js frappe.ui.form.on("My DocType", { // TODO: replace "My DocType" with your...
See also this comment: https://github.com/frappe/frappe/pull/19937#issuecomment-1426855243
Is it enough to just remove these three lines?
Please carefully verify that the problem and fix are valid, since this is my first contribution here and I don't really know what I'm doing. 😄
Looks like that one is already submitted ;)
@GangaManoj docs have been refactored to have a v12 and v13 folder for the respective version of ERPNext. Kindly update your PR to take this new structure into account.
> The [third commit](https://github.com/frappe/erpnext/pull/40659/commits/f3dbe12d7c7fad0409da08e745fe342a14459954) is entirely optional, and I probably should keep it private on my fork for now. Please let me know what to do. 🙏 Probably remove for...
For point (6) [we're thinking about](https://t.me/frappe_contributors/8236/8266) moving to [`.po` files](https://www.gnu.org/software/gettext/manual/html_node/PO-Files.html) instead of the existing `.csv` and [Transifex](https://www.transifex.com/open-source/) as a translation platform. One of the biggest reasons for .po files is...
@safwansamsudeen kindly add "before" and "after" screenshots, so it's easier for the reviewer to understand what you did.
@safwansamsudeen don't expect to get a review, as long as you don't bother to explain what you changed and why. ;)