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: add param letterhead to frappe.get_print #18989

Merged
merged 2 commits into from Nov 24, 2022

Conversation

barredterra
Copy link
Collaborator

@barredterra barredterra commented Nov 23, 2022

Until now, there was no way to specify the letterhead to be used by frappe.get_print.

no-docs

@barredterra barredterra requested a review from a team as a code owner November 23, 2022 18:24
@barredterra barredterra requested review from phot0n and removed request for a team November 23, 2022 18:24
@github-actions github-actions bot added the add-test-cases Add test case to validate fix or enhancement label Nov 23, 2022
@codecov
Copy link

codecov bot commented Nov 23, 2022

Codecov Report

Merging #18989 (5b84399) into develop (8486109) will decrease coverage by 0.04%.
The diff coverage is 100.00%.

❗ Current head 5b84399 differs from pull request most recent head d20a08a. Consider uploading reports for the commit d20a08a to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #18989      +/-   ##
===========================================
- Coverage    63.48%   63.43%   -0.05%     
===========================================
  Files          750      750              
  Lines        67618    67695      +77     
  Branches      6027     6027              
===========================================
+ Hits         42926    42944      +18     
- Misses       21230    21289      +59     
  Partials      3462     3462              
Flag Coverage Δ
server-mariadb 67.30% <100.00%> (-0.02%) ⬇️
server-postgres 67.30% <100.00%> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

@ankush ankush removed the add-test-cases Add test case to validate fix or enhancement label Nov 24, 2022
@ankush ankush merged commit 73f0256 into frappe:develop Nov 24, 2022
@barredterra barredterra deleted the param-letterhead branch November 24, 2022 09:08
@barredterra barredterra added the backport version-14-hotfix backport to version 14 label Nov 28, 2022
mergify bot pushed a commit that referenced this pull request Nov 28, 2022
* feat: add param letterhead to frappe.get_print

(cherry picked from commit 73f0256)
ankush pushed a commit that referenced this pull request Nov 29, 2022
* feat: add param letterhead to frappe.get_print

(cherry picked from commit 73f0256)

Co-authored-by: Raffael Meyer <14891507+barredterra@users.noreply.github.com>
frappe-pr-bot pushed a commit that referenced this pull request Nov 29, 2022
# [14.17.0](v14.16.0...v14.17.0) (2022-11-29)

### Bug Fixes

* Avoid `update_order` twice on Kanban load ([#19011](#19011)) ([#19013](#19013)) ([c6e2d16](c6e2d16))
* broken link for email tracking pixels ([#19030](#19030)) ([#19032](#19032)) ([ca520e0](ca520e0))
* Correctness in get_role_permissions and has_perm JS APIs (backport [#18877](#18877))  ([#18976](#18976)) ([cc2cd5f](cc2cd5f))
* default value options: sort ([#19019](#19019)) ([#19022](#19022)) ([3ec6d9c](3ec6d9c))
* discovery and styling issues in grid buttons ([a6bda7f](a6bda7f))
* handle "No Letterhead" in new print format builder ([#18990](#18990)) ([#18992](#18992)) ([1ad1cab](1ad1cab))
* horizontal scroll in rtl language ([906336e](906336e))
* only accept string values for `key` ([237a152](237a152))
* webform read only field not working ([#19026](#19026)) ([ce505ca](ce505ca))
* z-index for barcode and awesomeplete (backport [#18893](#18893)) ([#19003](#19003)) ([0191858](0191858))

### Features

* add param letterhead to frappe.get_print ([#18989](#18989)) ([#19034](#19034)) ([c887be5](c887be5))
stephenBDT pushed a commit to alias/frappe that referenced this pull request Nov 30, 2022
…#19034)

* feat: add param letterhead to frappe.get_print

(cherry picked from commit 73f0256)

Co-authored-by: Raffael Meyer <14891507+barredterra@users.noreply.github.com>
stephenBDT pushed a commit to alias/frappe that referenced this pull request Nov 30, 2022
# [14.17.0](frappe/frappe@v14.16.0...v14.17.0) (2022-11-29)

### Bug Fixes

* Avoid `update_order` twice on Kanban load ([frappe#19011](frappe#19011)) ([frappe#19013](frappe#19013)) ([c6e2d16](frappe@c6e2d16))
* broken link for email tracking pixels ([frappe#19030](frappe#19030)) ([frappe#19032](frappe#19032)) ([ca520e0](frappe@ca520e0))
* Correctness in get_role_permissions and has_perm JS APIs (backport [frappe#18877](frappe#18877))  ([frappe#18976](frappe#18976)) ([cc2cd5f](frappe@cc2cd5f))
* default value options: sort ([frappe#19019](frappe#19019)) ([frappe#19022](frappe#19022)) ([3ec6d9c](frappe@3ec6d9c))
* discovery and styling issues in grid buttons ([a6bda7f](frappe@a6bda7f))
* handle "No Letterhead" in new print format builder ([frappe#18990](frappe#18990)) ([frappe#18992](frappe#18992)) ([1ad1cab](frappe@1ad1cab))
* horizontal scroll in rtl language ([906336e](frappe@906336e))
* only accept string values for `key` ([237a152](frappe@237a152))
* webform read only field not working ([frappe#19026](frappe#19026)) ([ce505ca](frappe@ce505ca))
* z-index for barcode and awesomeplete (backport [frappe#18893](frappe#18893)) ([frappe#19003](frappe#19003)) ([0191858](frappe@0191858))

### Features

* add param letterhead to frappe.get_print ([frappe#18989](frappe#18989)) ([frappe#19034](frappe#19034)) ([c887be5](frappe@c887be5))
stephenBDT pushed a commit to alias/frappe that referenced this pull request Dec 1, 2022
…#19034)

* feat: add param letterhead to frappe.get_print

(cherry picked from commit 73f0256)

Co-authored-by: Raffael Meyer <14891507+barredterra@users.noreply.github.com>
stephenBDT pushed a commit to alias/frappe that referenced this pull request Dec 1, 2022
# [14.17.0](frappe/frappe@v14.16.0...v14.17.0) (2022-11-29)

### Bug Fixes

* Avoid `update_order` twice on Kanban load ([frappe#19011](frappe#19011)) ([frappe#19013](frappe#19013)) ([c6e2d16](frappe@c6e2d16))
* broken link for email tracking pixels ([frappe#19030](frappe#19030)) ([frappe#19032](frappe#19032)) ([ca520e0](frappe@ca520e0))
* Correctness in get_role_permissions and has_perm JS APIs (backport [frappe#18877](frappe#18877))  ([frappe#18976](frappe#18976)) ([cc2cd5f](frappe@cc2cd5f))
* default value options: sort ([frappe#19019](frappe#19019)) ([frappe#19022](frappe#19022)) ([3ec6d9c](frappe@3ec6d9c))
* discovery and styling issues in grid buttons ([a6bda7f](frappe@a6bda7f))
* handle "No Letterhead" in new print format builder ([frappe#18990](frappe#18990)) ([frappe#18992](frappe#18992)) ([1ad1cab](frappe@1ad1cab))
* horizontal scroll in rtl language ([906336e](frappe@906336e))
* only accept string values for `key` ([237a152](frappe@237a152))
* webform read only field not working ([frappe#19026](frappe#19026)) ([ce505ca](frappe@ce505ca))
* z-index for barcode and awesomeplete (backport [frappe#18893](frappe#18893)) ([frappe#19003](frappe#19003)) ([0191858](frappe@0191858))

### Features

* add param letterhead to frappe.get_print ([frappe#18989](frappe#18989)) ([frappe#19034](frappe#19034)) ([c887be5](frappe@c887be5))
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 13, 2022
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.

None yet

3 participants