Repository navigation
[ADD] om_aed_currency_symbol: UAE Dirham sign - #212
github-actions[bot] merged 1 commit into
Conversation
|
Hi @akshaymanoj87, thank you, this is very good work: every point of our review is addressed, and CI is green. One last thing, about the origin of the glyph. The README says that the glyph and the fonts were "created for this module by Odoo Mates", and the font files say "Odoo Mates. Licensed under LGPL-3". However, the Content Credentials (C2PA) embedded in Could you please:
Please amend your single commit so the PR stays one commit. Once that is in, the module is ready on our side; we will merge it a little later, together with the review of the rest of the batch. Thanks again! |
c29b246 to
ffd4837
Compare
ffd4837 to
b288798
Compare
|
/mergebot r+ |
|
Merging now. The branch is 4 commit(s) behind |
|
🎉 Merged, thank you @akshaymanoj87! Welcome to the contributors of Odoo Apps: this is your first merged pull request here, and we hope the first of many. Kudos for [ADD] om_aed_currency_symbol: UAE Dirham sign: it is on the |
What and why
Adds om_aed_currency_symbol for 20.0: shows the UAE Dirham sign (U+20C3) for the AED currency in the backend, PDF reports, website, portal and Point of Sale, using a small unicode-range font for devices that don't support the sign yet. It is a bridge until Odoo supports the sign natively.
This addresses the review feedback sent to the contributor by e-mail:
How to test
Versions
Branch 20.0. Backports to 18.0 and 19.0 will follow after this is merged.