Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge from upstream v13 #52

Merged
merged 153 commits into from Mar 7, 2022

Conversation

Alchez
Copy link

@Alchez Alchez commented Mar 2, 2022

ruthra-kumar and others added 30 commits January 27, 2022 20:51
(cherry picked from commit bf68f76)
(cherry picked from commit 5e2bfbd)
# Conflicts:
#	frappe/core/doctype/file/test_file.py
…#15614) (frappe#15812)

* fix(ux): show report button on too many writes error (frappe#15614)

* 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>
(cherry picked from commit bd8ac90)

* chore: space -> tabs

* test: skip on postgres

this doesn't work on v13 branch because writes check is completely ignored right now

Co-authored-by: Saqib Ansari <nextchamp.saqib@gmail.com>
Co-authored-by: Ankush Menat <ankush@frappe.io>
…5843)

* fix: dynamic progress-bar color

(cherry picked from commit c467708)
…racters

(cherry picked from commit af5e909)

# Conflicts:
#	frappe/core/doctype/doctype/doctype.py
(cherry picked from commit 85680cc)
(cherry picked from commit 749ace6)
- Empty values for link field is invalid and causes issue mentioned here frappe#15320
(cherry picked from commit 12c7f2a)
(cherry picked from commit 962a373)
gavindsouza and others added 28 commits February 10, 2022 18:31
fix(render): Check if valid file instead of path [v13]
(cherry picked from commit f0a10ef)
(cherry picked from commit a49a07e)
(cherry picked from commit d8fc396)
(cherry picked from commit f777700)
(cherry picked from commit a18703a)
(cherry picked from commit e21275f)
(cherry picked from commit 976c3be)
…fix/pr-15918

feat: Allow all users to customize reports of type Report Builder (backport frappe#15918)
(cherry picked from commit dce336f)
Co-authored-by: Wolfram Schmidt <wolfram.schmidt@phamos.eu>
…rappe#15832) (frappe#15942)

Co-authored-by: Pruthvi Patel <pruthvipatel145@gmail.com>
Co-authored-by: Sagar Vora <sagar@resilient.tech>
…rappe#15969) (frappe#15974)

Co-authored-by: Shariq Ansari <sharique.rik@gmail.com>
Co-authored-by: Shariq Ansari <30859809+shariquerik@users.noreply.github.com>
…fix-to-pre-release-for-13-21

chore: Merge branch 'version-13-hotfix' into 'version-13-pre-release'
…-release/pr-15982

fix: libsass issue with `min()` (backport frappe#15982)
(cherry picked from commit 1e4b397)
@commit-lint
Copy link

commit-lint bot commented Mar 2, 2022

Bug Fixes

Tests

  • add test cases for Data Export tool (cf1b4fa)
  • Add test for thumbnail creation of image without extn (57867ed)
  • added unit test (8dfd48c)
  • Update UI test for link field (34e258c)
  • Update test case for default value in link field (52604d9)

Features

  • add Attach Image in add_fetch (c369397)
  • add breadcrumbs in dashboard view (921ce1f)

Chore

  • Add typing, use better supported APIs (36f5833)
  • better error message (402d107)
  • Remove duplicate command (c55b012)

Code Refactoring

Contributors

ruthra-kumar, hrwX, surajshetty3416, gavindsouza, mergify[bot], shariquerik, nabinhait

Commit-Lint commands

You can trigger Commit-Lint actions by commenting on this PR:

  • @Commit-Lint merge patch will merge dependabot PR on "patch" versions (X.X.Y - Y change)
  • @Commit-Lint merge minor will merge dependabot PR on "minor" versions (X.Y.Y - Y change)
  • @Commit-Lint merge major will merge dependabot PR on "major" versions (Y.Y.Y - Y change)
  • @Commit-Lint merge disable will desactivate merge dependabot PR
  • @Commit-Lint review will approve dependabot PR
  • @Commit-Lint stop review will stop approve dependabot PR

@vjFaLk vjFaLk merged commit 755310c into ParsimonyGit:parsimony-production-v13 Mar 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet