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.

Sign up for GitHub

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

Jump to bottom

fix: Merge "Accounting Ledger" and "Accounts Receivable" in "View" button #27769

Merged
merged 3 commits into from
Oct 4, 2021
Merged

fix: Merge "Accounting Ledger" and "Accounts Receivable" in "View" button #27769

merged 3 commits into from
Oct 4, 2021

Conversation

Komal-Saraf0609
Copy link
Contributor

Created a new button "View" and moved "Accounting Ledger" and "Accounts Receivable".

Before:
image

After:
image

@ankush ankush added the squash Meant to tell reviewers that this PR should be squashed into a single commit while merging. label Oct 4, 2021
@ankush
Copy link
Member

ankush commented Oct 4, 2021

@Mergifyio backport version-13-hotfix

@frappe frappe deleted a comment from mergify bot Oct 4, 2021
@mergify
Copy link
Contributor

mergify bot commented Oct 4, 2021

Command backport version-13-hotfix: pending

backport version-13-hotfix is pending

@ankush ankush merged commit b483f17 into frappe:develop Oct 4, 2021
mergify bot pushed a commit that referenced this pull request Oct 4, 2021
@Komal-Saraf0609
fix: Merge "Accounting Ledger" and "Accounts Receivable" in "View" bu…
406acb2 
…tton (#27769)

* fix: Added a new button "View" and merged "Accounting Ledger" and "Accounts Receivable" into it

* fix: sider issues

* chore: dead code

(cherry picked from commit b483f17)
@mergify mergify bot mentioned this pull request Oct 4, 2021
Merged
@mergify
Copy link
Contributor

mergify bot commented Oct 4, 2021

Command backport version-13-hotfix: success

Backports have been created

ankush pushed a commit that referenced this pull request Oct 4, 2021
@mergify @Komal-Saraf0609
fix: Merge "Accounting Ledger" and "Accounts Receivable" in "View" bu…
21a5498 
…tton (#27769) (#27771)

* fix: Added a new button "View" and merged "Accounting Ledger" and "Accounts Receivable" into it

* fix: sider issues

* chore: dead code

(cherry picked from commit b483f17)

Co-authored-by: Komal-Saraf0609 <81952590+Komal-Saraf0609@users.noreply.github.com>
asoral pushed a commit to asoral/erpnext that referenced this pull request Nov 12, 2021
@Komal-Saraf0609
fix: Merge "Accounting Ledger" and "Accounts Receivable" in "View" bu…
0891fdd 
…tton (frappe#27769)

* fix: Added a new button "View" and merged "Accounting Ledger" and "Accounts Receivable" into it

* fix: sider issues

* chore: dead code
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 18, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ankush ankush ankush approved these changes

@deepeshgarg007 deepeshgarg007 Awaiting requested review from deepeshgarg007

@nextchamp-saqib nextchamp-saqib Awaiting requested review from nextchamp-saqib

Assignees

@umairsy umairsy

@harshit-30 harshit-30

Labels
squash Meant to tell reviewers that this PR should be squashed into a single commit while merging.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Komal-Saraf0609 @ankush @umairsy @harshit-30