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

feat: let users unlock stuck documents #24782

Merged
merged 6 commits into from
Feb 6, 2024

Conversation

ankush
Copy link
Member

@ankush ankush commented Feb 6, 2024

  • fix: better file locking
  • fix: Auto delete very old document locks
  • fix!: Accept flat arguments for server_action
  • feat: support primary_action for frappe.throw
  • feat: let users unlock stuck documents

PS: This PR contains a breaking change, don't just backport directly

locks older than 12 hours are most likely from dead processes. They can be (mostly) safely ignored.
@ankush ankush requested review from a team and surajshetty3416 and removed request for a team February 6, 2024 07:16
@ankush ankush force-pushed the file_locking_changes branch 2 times, most recently from 1c16c39 to 625149c Compare February 6, 2024 07:21
frappe/utils/file_lock.py Outdated Show resolved Hide resolved
@ankush ankush force-pushed the file_locking_changes branch 2 times, most recently from 7f7b928 to f0a53dd Compare February 6, 2024 07:31
Co-authored-by: Akhil Narang <me@akhilnarang.dev>
ankush added a commit to frappe/erpnext that referenced this pull request Feb 6, 2024
args are now flat, no need to accept them as dict

ref: frappe/frappe#24782
@ankush ankush merged commit 7b83fce into frappe:develop Feb 6, 2024
24 checks passed
@ankush ankush deleted the file_locking_changes branch February 6, 2024 07:49
ankush added a commit to ankush/hrms that referenced this pull request Feb 6, 2024
We accept flat argument instead of `args` now. Ref: frappe/frappe#24782
ankush added a commit to frappe/erpnext that referenced this pull request Feb 6, 2024
args are now flat, no need to accept them as dict

ref: frappe/frappe#24782
ankush added a commit to ankush/hrms that referenced this pull request Feb 6, 2024
We accept flat argument instead of `args` now. Ref: frappe/frappe#24782
ruchamahabal pushed a commit to frappe/hrms that referenced this pull request Feb 6, 2024
We accept flat argument instead of `args` now. Ref: frappe/frappe#24782
mergify bot pushed a commit to frappe/hrms that referenced this pull request Feb 6, 2024
We accept flat argument instead of `args` now. Ref: frappe/frappe#24782

(cherry picked from commit 80e6193)
mergify bot pushed a commit to frappe/hrms that referenced this pull request Feb 6, 2024
We accept flat argument instead of `args` now. Ref: frappe/frappe#24782

(cherry picked from commit 80e6193)

# Conflicts:
#	hrms/hr/doctype/interview/interview.py
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 22, 2024
@ankush ankush added backport version-14-hotfix backport to version 14 backport version-15-hotfix Backport the PR to v15 labels Mar 4, 2024
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 backport version-15-hotfix Backport the PR to v15
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants