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: Added email bodyType to smtp action (#22749) #23078
Conversation
PR Summary
|
/ok-to-test |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/4914516065. |
/ok-to-test |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/4917082138. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/4917082138.
To know the list of identified flaky tests - Refer here |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/4917082138. |
1 similar comment
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/4917082138. |
app/server/appsmith-server/src/main/java/com/appsmith/server/migrations/DatabaseChangelog2.java
Outdated
Show resolved
Hide resolved
app/server/appsmith-server/src/main/java/com/appsmith/server/migrations/DatabaseChangelog2.java
Outdated
Show resolved
Hide resolved
...-server/src/main/java/com/appsmith/server/migrations/db/ce/Migration010AddEmailBodyType.java
Outdated
Show resolved
Hide resolved
app/server/appsmith-plugins/smtpPlugin/src/main/java/com/external/plugins/SmtpPlugin.java
Show resolved
Hide resolved
/ok-to-test |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/4933491659. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/4933491659.
To know the list of identified flaky tests - Refer here |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/4933491659. |
/build-deploy-preview |
Deploying Your Preview: https://github.com/appsmithorg/appsmith/actions/runs/4936352472. |
Deploy-Preview-URL: https://ce-23078.dp.appsmith.com |
HI @sanveer-osahan |
Description
Added a new field bodyType to SMTP action.
The bodyType field specifies the content-type for email body. Users can choose between 2 options
text/plain
)text/html
)PR fixes following issue(s)
Fixes #22749
Type of change
Testing
How Has This Been Tested?
Test Plan
Issues raised during DP testing
Checklist:
Dev activity
QA activity:
Test Plan Approved
label after Cypress tests were reviewedTest Plan Approved
label after JUnit tests were reviewed