-
Notifications
You must be signed in to change notification settings - Fork 122
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
Store notification content as binary to support unicode #280
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #280 +/- ##
============================================
- Coverage 17.11% 17.00% -0.12%
Complexity 180 180
============================================
Files 53 53
Lines 3027 3047 +20
Branches 332 333 +1
============================================
Hits 518 518
- Misses 2467 2487 +20
Partials 42 42 ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
....email.mgt/src/main/java/org/wso2/carbon/email/mgt/store/dao/AppNotificationTemplateDAO.java
Outdated
Show resolved
Hide resolved
....email.mgt/src/main/java/org/wso2/carbon/email/mgt/store/dao/AppNotificationTemplateDAO.java
Outdated
Show resolved
Hide resolved
...gt/org.wso2.carbon.email.mgt/src/main/java/org/wso2/carbon/email/mgt/util/I18nEmailUtil.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot reviewed 5 out of 6 changed files in this pull request and generated 2 suggestions.
Files not reviewed (1)
- pom.xml: Language not supported
Proposed changes in this pull request
$subject. Related to,
When should this PR be merged
After merging,
Follow up actions
When updating product-is versions, both the framework change and this should be update together.