This is reusing normal cache which doesn't contain half the things

(cherry picked from commit 236ab8dbed)

(cherry picked from commit 8541fa4cd3)

* feat: review link for Custom, build and NavBar

* feat: review link for Custom, build and NavBar

* feat: review link for Custom, build and NavBar

* feat: review link for Custom, build and NavBar

* feat: review link for Custom, build and NavBar

* feat: review link for Custom, build and NavBar

* chore: linter json

* fix: replace Block Module By Module Profile

* chore: cypress test

* chore: reset package.json to develop

* refactor: workspace link, system logs section

- removed workspace listview link, visiting list view isn't required for
  practically all the tasks
- Frappe Logs -> System Logs

[skip ci]

(cherry picked from commit df9e8c62dd)

Co-authored-by: HENRY Florian <florian.henry@open-concept.pro>

[skip ci]

(cherry picked from commit 7b646ccec5)

Co-authored-by: Shariq Ansari <30859809+shariquerik@users.noreply.github.com>

feat: add lang to safe globals (backport #18011)

(cherry picked from commit d0b753a25d)

(cherry picked from commit 836fd6ef78)

Not required on stable branches.

inlieu of event_streaming app

* Added button to collapse tree root node

* Removed unneccessary space

* Collapse all expanded nodes

* style: prettier

* refactor: load first level root nodes by default

Co-authored-by: Ankush Menat <ankush@frappe.io>
(cherry picked from commit e8b0b991ab)

Co-authored-by: Nikhil Kothari <nik.kothari22@live.com>

Rerunning this isn't required and might cause unexepcted behaviour

[skip ci]

(cherry picked from commit 6a24748f9a)

Co-authored-by: Shariq Ansari <30859809+shariquerik@users.noreply.github.com>

* perf: reorder condition to avoid redis call

* test: basic perf tests

(cherry picked from commit f5b8e5f015)

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

When this checkbox is checked, while uploading docs using fileuploader
the default visibility will be set to "public".

At present all uploaded attachments are private, however in following
cases it makes sense to set default as public:

1. Blog post
2. Item catalog
3. Website item

Considering this we decided to add this configurtion so developers can
specify sane default while user still has control over it using
"Customize form"

Enabled this for "blog post" doctype

Unrelatd change: consistent "form settings" section on DocType and customize
form

(cherry picked from commit e8af6b2c16)

(cherry picked from commit bde4e44cf4)

Co-authored-by: Suraj Shetty <13928957+surajshetty3416@users.noreply.github.com>

chore: update fr translation (backport #17950)

(cherry picked from commit aac5e3d3c1)

[skip ci]

* feat(bench): add new bench command for add user

(cherry picked from commit 40f54d04b7)

* test: simplify user addition test

(cherry picked from commit 6ec5c05e4e)

Co-authored-by: Alaa Alsalehi <alaadiaa@gmail.com>
Co-authored-by: Ankush Menat <ankush@frappe.io>

* test: undo custom field creation

(cherry picked from commit 41af9621cd)

* test: Use FrappeTestCase everywhere

(cherry picked from commit 3e2d2a703a)

# Conflicts:
#	frappe/printing/doctype/print_format/test_print_format.py
#	frappe/tests/test_commands.py
#	frappe/tests/test_config.py
#	frappe/tests/test_oauth20.py
#	frappe/tests/test_translate.py
#	frappe/tests/test_utils.py

* fix: dont use sendmail with now=True

(cherry picked from commit 1c9ea57279)

* test: skip badly written test temporarily

(cherry picked from commit 613d8472b4)

* chore: conflicts

Co-authored-by: Ankush Menat <ankush@frappe.io>
Co-authored-by: phot0n <ritwikpuri5678@gmail.com>

* feat: remove subtree

* test: remove_subtree

(cherry picked from commit d05ed27a01)

Co-authored-by: Raffael Meyer <14891507+barredterra@users.noreply.github.com>

- Revert https://github.com/frappe/frappe/pull/17390 since it used to set this.value before triggering change events... due to this this.last_value & this.value was having same value in change events of some control (eg. link field)
- set_input has the responsibility to set this.value and this.last_value properly.

(cherry picked from commit 2e91e92227)

fix: permissions dont refresh when switching doc (backport #17884)

(cherry picked from commit 722d57637f)

Grid permissions aren't refreshed when switching docs.

Setup permissions like this:

1. Normal perm - read
2. If owner - read + write + others
3. Open a normal doc -> is read only so grid is read only.
4. Now go back and open a created doc -> grid will be read only or empty
   (if no rows)

fix: make grid.perm a computed property that refers to form instead of
duplicating this.

ref: ISS-22-23-01733
(cherry picked from commit 80d264a7b4)

(cherry picked from commit 63a60c6457)

Co-authored-by: Raffael Meyer <14891507+barredterra@users.noreply.github.com>

TypeError: 'TopBarItem' object does not support item assignment
(cherry picked from commit 9358655ae2)

Co-authored-by: Faris Ansari <netchampfaris@users.noreply.github.com>

(cherry picked from commit cdb0732646)

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

(cherry picked from commit d17fd03958)

(cherry picked from commit 78ab3d8429)

(cherry picked from commit 5ca89e0dad)

fix: fetch email domain values for email account (backport #17883)

Also removed  tag from domain and service labels in email account doctype

(cherry picked from commit 141693a75b)

(cherry picked from commit c450058a37)

* chore: remove dead code related to fetching the same

(cherry picked from commit a3cd00a3ed)

chore: remove old code where User Permissions were set in `tabDefaultValue` (backport #17294)

(cherry picked from commit c5face41b3)

(cherry picked from commit 2866721a78)

# Conflicts:
#	frappe/tests/test_translate.py

(cherry picked from commit 63e760e3ad)

(cherry picked from commit 2b57e77f29)

color field causing error in view mode (fixed)

(cherry picked from commit 7d245268ea)

(cherry picked from commit fb8ba3d881)

(cherry picked from commit 02da933c16)

(cherry picked from commit 4b4d66d2a3)