Co-authored-by: hrwx <himanshuwarekar@yahoo.com>
Co-authored-by: Raffael Meyer <14891507+barredterra@users.noreply.github.com>
Co-authored-by: Ankush Menat <ankush@frappe.io>
(cherry picked from commit b0c1e400ea)

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

(cherry picked from commit 458757708f)

(cherry picked from commit 930f8c34ed)

(cherry picked from commit 36bc9d3142)

* test(UI): use non-admin user for tests

(cherry picked from commit f07bc3b369)

# Conflicts:
#	.github/workflows/ui-tests.yml

* test: fix tests using admin account

(cherry picked from commit a6b341bd59)

# Conflicts:
#	cypress/integration/dashboard_links.js

* test: add test role to user

(cherry picked from commit 0896873c8a)

* chore: conflicts

* test: fix broken test

Not sure why there's test on doctype "created by". Makes no sense.

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

Cypress supports this natively now!

(cherry picked from commit c400e18979)

chore: run cypress migrator

fix: specPattern should be an array

test: correct intercept URL

This is breaking change in cypress 7.0

test: make list view test rerunnable

test: redo undo tests and add compare doc

test: scroll to action button

chore: drop flaky tests

fix: remove scroll behaviour

fix: clear filters before running count test

test: input delay causes flake
(cherry picked from commit e397b27d29)

* 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>

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 2181233c75)

* feat: custom __link_optins

* chore: linter fixes

* test: add ui test for custom link option

(cherry picked from commit 1aa3f75d4c)

Co-authored-by: Himanshu <himanshuwarekar@yahoo.com>

(cherry picked from commit 42a3fe0fa2)

(cherry picked from commit d9c3c03522)

(cherry picked from commit db2e898397)

(cherry picked from commit b01929405d)

# Conflicts:
#	frappe/contacts/doctype/gender/gender.json
#	frappe/contacts/doctype/salutation/salutation.json

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

Use colored text instead

(cherry picked from commit 6ef41e03e3)

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

(cherry picked from commit 40f27f908a)

* test: flaky tests due to uncleared filters

* perf: remove duplicate code in assets

Remove controls which are separately bundled too, saves ~1.1MB of
network transfer.

Co-authored-by: Suraj Shetty <surajshetty3416@gmail.com>

* fix(model)!: Match labels in meta & model

Labels for fields have been inconsistent. In standard filters, doc.name
would be labelled as "Name" while it's "ID" in the edit filters option.
Similarly for other fields, the same view will address the same fields
differently. This PR aims to get rid of such inconsistencies.

Changes:

* [model] Change label ID for name to 'Name'
* [meta] Match labels of owner, modified, modified_by
* [meta] Add labels for more fields - from model.js

* fix!: get_label to fetch labels as Desk does

* fix!(label): Refer to name field as 'ID' over 'Name'

* test: Use 'ID' label instead of 'Name'

Mandatory fields are highlighted red on page load. It should only highlight if we save the document.

**Before:** Form

https://user-images.githubusercontent.com/30859809/168791201-049c1de2-b7f5-4dff-bf40-093ff96d4510.mov

**After:** Form

https://user-images.githubusercontent.com/30859809/168791164-45e2289b-2d2c-4ebb-8b4d-cd2932140572.mov

**Before:** Dialog

https://user-images.githubusercontent.com/30859809/168791207-800ba9e5-b4eb-419f-8a99-f94916c21c79.mov

**After:** Dialog

https://user-images.githubusercontent.com/30859809/168791210-f2761d23-3a60-40c2-8aff-2d2cebbd77fa.mov

Resolves https://github.com/frappe/frappe/issues/16866

- Fixes [Issue/15324](https://github.com/frappe/frappe/issues/15324)
- When selecting a value for Link Field, the English text was set as link field value.
- This PR aims to fix it by setting the translated text as link field value on selection.

Todo:

- [x] Show translated text in the select dropdown
- [x] Set translated value to Link field on select
- [x] Show original value when link field is in focus
- [x] Add option to toggle this behaviour

### Behaviour

- Link field loses focus: show the translated name.
- Link field is focused
    - If old value is present
        - If options **are** one of [Role, DocType]: show the translated name
        - Else: show the name (untranslated) to enable search in untranslated values
    - Else: show what the user typed (untranslated) to enable search in untranslated values
- Value is selected: link field loses focus

## Demo

### Link to UOM in a custom Item DocType

The UOM names are in english, so the search needs to happen in english. When possible, the translation is displayed.

#### Before

https://user-images.githubusercontent.com/14891507/156415248-e5e80d05-53dc-4ca8-89c7-998986ff6e99.mov

#### After

https://user-images.githubusercontent.com/14891507/156410386-a874430c-f340-43ed-9c3a-92e8d4d50fc9.mov

### Link to DocType in Customize Form

The DocType names get translated before being searched. This is a preexisting hack in the framework for DocType and Role. In this case, we can search in the translations.

#### Before

https://user-images.githubusercontent.com/14891507/156414648-8e505f8c-9dee-4358-8182-3b358c28bb62.mov

#### After

https://user-images.githubusercontent.com/14891507/156411881-c4ca22e1-1397-4e13-9768-5e16b72f8d6d.mov

https://docs.erpnext.com/docs/v13/user/manual/en/customize-erpnext/customize-form/edit?wiki_page_patch=fdafee2715

This reverts commit d43d9594f0.

* chore: updated the warning under autoincrement autoname

Unable to find the cause of the flakiness of the test. Disabling it for now.

Also, the form_tour test is full of `wait` which is bad anyway.

Disable form tour tests