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

[Notification Template API] API Implementation #629

Merged
merged 23 commits into from
Oct 24, 2024

Conversation

RushanNanayakkara
Copy link
Contributor

@RushanNanayakkara RushanNanayakkara commented Jul 10, 2024

Purpose

Introduce Notification Template API which supports template operations for all available notification channels. (Email, SMS)

Release note

${Product} now provides the support to manage and update your notification templates through a single API. This API introduces support for SMS template management and supports email template management as well.

Documentation

Automation tests

  • Unit tests

    Code coverage information

  • Integration tests

    Details about the test cases and coverage

Samples

Provide high-level details about the samples related to this feature

Related Issues

Related PRs

Learning

Describe the research phase and any blog posts, patterns, libraries, or add-ons you used to solve the problem.

pom.xml Outdated
<carbon.identity.framework.version>7.5.0</carbon.identity.framework.version>
<maven.findbugsplugin.version>3.0.5</maven.findbugsplugin.version>
<findsecbugs-plugin.version>1.12.0</findsecbugs-plugin.version>
<maven.checkstyleplugin.excludes>**/gen/**/*</maven.checkstyleplugin.excludes>
<identity.event.handler.version>1.8.19</identity.event.handler.version>
<identity.event.handler.version>1.9.5-SNAPSHOT</identity.event.handler.version>
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The snapshot versions need to be bumped after dependent PRs are merged. The snapshot values will be replaced with new release versions.

Co-authored-by: AmshikaH <66556626+AmshikaH@users.noreply.github.com>
Avarjana
Avarjana previously approved these changes Oct 23, 2024
@jenkins-is-staging
Copy link

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/11486880189

@jenkins-is-staging
Copy link

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/11486880189
Status: failure

@jenkins-is-staging
Copy link

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/11489863048

@jenkins-is-staging
Copy link

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/11489863048
Status: success

Copy link

@jenkins-is-staging jenkins-is-staging left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving the pull request based on the successful pr build https://github.com/wso2/product-is/actions/runs/11489863048

@RushanNanayakkara RushanNanayakkara merged commit 0f81de6 into wso2:master Oct 24, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants