This reverts commit 79f5a6b640.

fix: communication missing from form timeline

fix!: remove unnecessary array transformation in request args

test(User Type): add select perm doctypes

* feat: log errors on too many writes to ease debugging

* fix(ux): extract and show which app/file/function caused too many writes

* fix: postgres error checking assumes exception is postgres exception

* fix: better default for skipping frames

Typically stack is like: some_app -> your_function -> this function. So last 2 frames need to be skipped.

* fix: postgres error checking assumes exception is postgres exception

* fix: better default for skipping frames

Typically stack is like: some_app -> your_function -> this function. So last 2 frames need to be skipped.

* refactor: show Report button instead of logging

* revert: unnecessary new functionality

* test: assert exact exception

Co-authored-by: Ankush Menat <ankush@frappe.io>

fix: New Print Format Builder fixes

added missing translation in DocType User

refactor!: remove post and post comment doctypes

feat: add Attach Image in add_fetch

feat: signature field in webform

fix: translate title in msgprint

test: xlsxutils

`key: ['value', 'value2']` is turned into `key: 'value'` for no reason

fix!: Misc fixes

fix: Add patch to remove chat related doctypes

add .gitkeep file so that public folder is committed to git this is
needed because if public doesn't exist, bench build doesn't
symlink the public folder to sites/assets

Added translation connected to: https://github.com/frappe/frappe/pull/15669

* Avoid double save if possible - seemed like it was added mistakenly,
  didnt find enough evidence for it's addition
* Avoid json.loads breaking if "" was passed for attachments

Prior to this, no tracebacks were logged and the body of the log only
had what was supposed to be the title