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

fix: Use frm instead of this #1990

Merged
merged 2 commits into from
Jul 21, 2024
Merged

Conversation

marination
Copy link
Contributor

this refers to the window object that does not have a doctype attribute

On v14, this seems to not be initialized and results in:
Screenshot 2024-07-19 at 5 59 36 PM

@marination marination requested a review from ruchamahabal as a code owner July 19, 2024 16:00
@ruchamahabal ruchamahabal merged commit 82dd597 into frappe:develop Jul 21, 2024
4 checks passed
mergify bot pushed a commit that referenced this pull request Jul 21, 2024
(cherry picked from commit 82dd597)
mergify bot pushed a commit that referenced this pull request Jul 21, 2024
(cherry picked from commit 82dd597)
ruchamahabal pushed a commit that referenced this pull request Jul 21, 2024
(cherry picked from commit 82dd597)

Co-authored-by: Marica <[email protected]>
ruchamahabal pushed a commit that referenced this pull request Jul 21, 2024
(cherry picked from commit 82dd597)

Co-authored-by: Marica <[email protected]>
frappe-pr-bot pushed a commit that referenced this pull request Jul 27, 2024
## [14.29.1](v14.29.0...v14.29.1) (2024-07-27)

### Bug Fixes

* Use `frm` instead of `this` (backport [#1990](#1990)) ([#1994](#1994)) ([90721c4](90721c4))
frappe-pr-bot pushed a commit that referenced this pull request Jul 27, 2024
# [15.25.0](v15.24.0...v15.25.0) (2024-07-27)

### Bug Fixes

* **FnF:** ignore GL entry links & notify update on payment submission/cancellation ([8bbe23a](8bbe23a))
* skip Bank Entry for withheld salary slips ([ae7deb8](ae7deb8))
* Use `frm` instead of `this` (backport [#1990](#1990)) ([#1995](#1995)) ([5a2dcf2](5a2dcf2))
* validate duplicate salary withholdings ([107362b](107362b))

### Features

* add 'Withheld' status to Salary Slip ([609e71d](609e71d))
* add Salary Withholding doctypes ([0987460](0987460))
* add withholding cycle reference to salary slip & enable no-copy for read-only fields ([f390572](f390572))
* btn in Payroll Entry to Release Withheld Salaries ([6a6e430](6a6e430))
* dashboard connection for Salary Withholding from employee master ([1604939](1604939))
* fetch employees with withheld salaries in payroll entry ([cad5473](cad5473))
* link bank entry in salary withholdings ([387bcd7](387bcd7))
* populate withholding cycles in child table and set To Date ([80e60cd](80e60cd))
* pull withheld salary slips in FnF ([6b40f56](6b40f56))
* set withholding doc ref & status in salary slip ([390fc29](390fc29))
* status field in salary withholding based on withholding status for all cycles ([4a3f912](4a3f912))
* update salary withholding payment status on JV submission/cancellation ([fa5f179](fa5f179))
* update withholding status in payroll employee table ([a8695aa](a8695aa))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants