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(FnF): don't fetch outstanding statements on every save #797

Merged
merged 4 commits into from
Aug 23, 2023

Conversation

niyazrazak
Copy link
Contributor

The user cannot leave the receivables tables empty in the doctype. With each save, loans and employee advances are added to the table.

@ruchamahabal ruchamahabal changed the title chore: minor logic update in the full and final statement fix(FnF): don't fetch outstanding statements on every save Aug 23, 2023
@mergify mergify bot merged commit 266f270 into frappe:develop Aug 23, 2023
5 checks passed
@niyazrazak
Copy link
Contributor Author

Thank you @ruchamahabal

mergify bot pushed a commit that referenced this pull request Aug 23, 2023
The user cannot leave the receivables tables empty in the doctype. With each save, loans and employee advances are added to the table.

(cherry picked from commit 266f270)
ruchamahabal pushed a commit that referenced this pull request Aug 23, 2023
…797) (#816)

Co-authored-by: NIYAZ RAZAK <76736615+niyazrazak@users.noreply.github.com>
frappe-pr-bot pushed a commit that referenced this pull request Aug 23, 2023
## [14.10.2](v14.10.1...v14.10.2) (2023-08-23)

### Bug Fixes

* Allow float for `Maximum Leave Allocation Allowed` (backport [#789](#789)) ([#800](#800)) ([b931037](b931037))
* allow optional tax component addition & removal (backport [#814](#814)) ([#815](#815)) ([186e882](186e882))
* **FnF:** don't fetch outstanding statements on every save (backport [#797](#797)) ([#816](#816)) ([3664517](3664517))
* **Shift Type:** rename misleading Enable Entry/Exit Grace Period checkbox label (backport [#784](#784)) ([#799](#799)) ([f3a11e8](f3a11e8))
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