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(RFQ): optionally send document print #36363

Merged
merged 2 commits into from
Aug 22, 2023

Conversation

barredterra
Copy link
Collaborator

@barredterra barredterra commented Jul 27, 2023

Depends on frappe/frappe#21837

This PR allows, for each supplier, to attach the individual printout of the RFQ to the email. The print is in the supplier's language and uses the specified letterhead.

Bildschirmfoto 2023-07-27 um 19 49 33

Bildschirmfoto 2023-07-27 um 19 49 41

no-docs

(will add docs after merge)

@github-actions github-actions bot added the needs-tests This PR needs automated unit-tests. label Jul 27, 2023
@codecov
Copy link

codecov bot commented Jul 27, 2023

Codecov Report

Merging #36363 (d1e542c) into develop (9db8769) will decrease coverage by 0.45%.
Report is 4 commits behind head on develop.
The diff coverage is 0.00%.

❗ Current head d1e542c differs from pull request most recent head 0eeffa6. Consider uploading reports for the commit 0eeffa6 to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #36363      +/-   ##
===========================================
- Coverage    64.70%   64.25%   -0.45%     
===========================================
  Files          791      785       -6     
  Lines        61572    60969     -603     
===========================================
- Hits         39838    39174     -664     
- Misses       21734    21795      +61     
Files Changed Coverage Δ
...ype/request_for_quotation/request_for_quotation.py 53.84% <0.00%> (-1.87%) ⬇️

... and 52 files with indirect coverage changes

@barredterra barredterra added backport version-14-hotfix backport to version 14 and removed needs-tests This PR needs automated unit-tests. labels Aug 1, 2023
@s-aga-r s-aga-r merged commit 4b75b6f into frappe:develop Aug 22, 2023
12 checks passed
mergify bot pushed a commit that referenced this pull request Aug 22, 2023
@barredterra barredterra deleted the rfq-attach-print branch August 22, 2023 06:57
deepeshgarg007 pushed a commit that referenced this pull request Sep 2, 2023
feat(RFQ): optionally send document print (#36363)
frappe-pr-bot pushed a commit that referenced this pull request Sep 6, 2023
# [14.38.0](v14.37.1...v14.38.0) (2023-09-06)

### Bug Fixes

* account payable currency and value ([#36859](#36859)) ([98c2640](98c2640))
* added ignore_user_permissions to parent field of tree doctypes ([451cc7b](451cc7b))
* added validation for unique serial numbers in pos invoice ([#36302](#36302)) ([1192736](1192736))
* allow payment_account of loan repayment to be edited ([#36948](#36948)) ([09e2f24](09e2f24))
* ask for asset related accounts only when needed (backport [#36960](#36960)) ([#36971](#36971)) ([58163d5](58163d5))
* calcuate received/paid amount on rate change in PE ([0a63266](0a63266))
* deduplicate gain/loss JE creation for journals as payment ([9168b3b](9168b3b))
* difference amount in UI should not be calculated ([9bc2b41](9bc2b41))
* fetch discount amount for gle in base currency ([a8b5880](a8b5880))
* ignore mandatory fields while saving WO (backport [#36954](#36954)) ([#36970](#36970)) ([c125dea](c125dea))
* index error on Receivable report based on payment terms ([#36963](#36963)) ([e3d64fc](e3d64fc))
* invalid gain/loss JE created on base currency Expense Claim ([068f1b5](068f1b5))
* only show "Unreconcile" if reconciled ([61752ac](61752ac))
* prorate factor for subscription plan ([#36953](#36953)) ([fc79af5](fc79af5))
* reduce threshold for bg job fn ([5a226a8](5a226a8))
* remove withholding category from common fields ([18f8f7f](18f8f7f))
* Set the default filter in All Trends Report ([132957f](132957f))
* Set the default filter in All Trends Report ([420536c](420536c))
* when create doc from item dashboard default uom (buying or selling) is not correctly selected (backport [#36892](#36892)) ([#36928](#36928)) ([5c8bee0](5c8bee0))

### Features

* **RFQ:** optionally send document print ([#36363](#36363)) ([345c608](345c608))
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 6, 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants