8000 perf: use cached workflow doc for transitions by ankush · Pull Request #21300 · frappe/frappe · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
8000

perf: use cached workflow doc for transitions #21300

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

Merged
merged 2 commits into from
Jun 9, 2023

Conversation

ankush
Copy link
Member
@ankush ankush commented Jun 9, 2023

No description provided.

@github-actions github-actions bot added the add-test-cases Add test case to validate fix or enhancement label Jun 9, 2023
@ankush ankush removed the add-test-cases Add test case to validate fix or enhancement label Jun 9, 2023
@ankush ankush changed the title perf: faster workflow transitions perf: use cached workflow doc for transitions Jun 9, 2023
@ankush ankush marked this pull request as ready for review June 9, 2023 07:13
@ankush ankush requested review from a team and phot0n and removed request for a team June 9, 2023 07:13
@ankush ankush merged commit 774a75d into frappe:develop Jun 9, 2023
@ankush ankush deleted the faster_workflow_transition branch June 9, 2023 07:13
@ankush ankush added the backport version-14-hotfix backport to version 14 label Jun 9, 2023
mergify bot pushed a commit that referenced this pull request Jun 9, 2023
* fix: expire `RECORDER_INTERCEPT_FLAG` in one hour

Leaving this running in prod can be dangerous.

* perf: Use cached doc in workflows

(cherry picked from commit 774a75d)

# Conflicts:
#	frappe/recorder.py
ankush added a commit that referenced this pull request Jun 9, 2023
* perf: use cached workflow doc for transitions (#21300)

* fix: expire `RECORDER_INTERCEPT_FLAG` in one hour

Leaving this running in prod can be dangerous.

* perf: Use cached doc in workflows

(cherry picked from commit 774a75d)

# Conflicts:
#	frappe/recorder.py

* conflicts

---------

Co-authored-by: Ankush Menat <ankush@frappe.io>
frappe-pr-bot pushed a commit that referenced this pull request Jun 13, 2023
# [14.39.0](v14.38.3...v14.39.0) (2023-06-13)

### Bug Fixes

* allow video files to be uploaded ([9c43e40](9c43e40))
* Clear hash once the scrolling is done ([c89a390](c89a390))
* consider default value for content in update_workspace2 patch ([#21258](#21258)) ([#21286](#21286)) ([fa7c114](fa7c114))
* currency precision formatter ([#21293](#21293)) ([#21295](#21295)) ([c5fa12d](c5fa12d))
* error in workspace patch ([#21335](#21335))  ([b452c63](b452c63))
* handle fixture syncing for missing doctypes ([cad99f1](cad99f1))
* improve delete_contact_and_address ([#20381](#20381)) ([#21280](#21280)) ([fcb614e](fcb614e))
* New Dashboard Chart throws TypeError: format requires a mapping (backport [#21275](#21275)) ([#21299](#21299)) ([0a5bec5](0a5bec5)), closes [#21250](#21250)
* no role assigned message when role profile is passed ([#21342](#21342)) ([#21352](#21352)) ([e23503b](e23503b))
* Pass URL hash separately ([ef47548](ef47548))
* reuse redis connection if not rq auth ([#21336](#21336)) ([2748d9c](2748d9c))
* skip custom script syncing for missing doctypes ([12b8dac](12b8dac))
* sync importable doctype before documents ([#21131](#21131)) ([#21327](#21327)) ([0852b37](0852b37))
* virtual fields in child tables not displaying ([#20528](#20528)) ([#21329](#21329)) ([232f217](232f217))

### Features

* Scroll to field via URL ([604c4a3](604c4a3))

### Performance Improvements

* use cached workflo
73D4
w doc for transitions (backport [#21300](#21300)) ([#21301](#21301)) ([b6fb8a2](b6fb8a2))
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 24, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport version-14-hotfix backport to version 14
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant
0