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

Add capability to switch between storing notification content as binary data or text data #21784

Merged
merged 1 commit into from
Nov 21, 2024

Conversation

darshanasbg
Copy link
Contributor

@darshanasbg darshanasbg commented Nov 21, 2024

@jenkins-is-staging
Copy link
Contributor

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

@darshanasbg
Copy link
Contributor Author

darshanasbg commented Nov 21, 2024

Notification template related tests passed locally.. Thus continue with merging, in order to avoid having complications in the dependency updater.

[INFO] Tests run: 255, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 207.209 s - in TestSuite
[INFO]
[INFO] Results:
[INFO]
[INFO] Tests run: 255, Failures: 0, Errors: 0, Skipped: 0

@darshanasbg darshanasbg merged commit fb59da0 into wso2:master Nov 21, 2024
1 check passed
@jenkins-is-staging
Copy link
Contributor

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

@darshanasbg
Copy link
Contributor Author

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

Failing email otp test since it use its own deployment toml file.

2024-11-21T15:55:12.9235323Z [INFO] Results:
2024-11-21T15:55:12.9235854Z [INFO] 
2024-11-21T15:55:12.9236292Z [ERROR] Failures: 
2024-11-21T15:55:12.9238106Z [ERROR] org.wso2.identity.integration.test.EmailOTPTestCase.testEmailOTPAuthentication(org.wso2.identity.integration.test.EmailOTPTestCase)
2024-11-21T15:55:12.9241348Z [ERROR]   Run 1: EmailOTPTestCase.testEmailOTPAuthentication:196 Error in redirection to email OTP authentication page for user: testuser1 expected:<true> but was:<false>
2024-11-21T15:55:12.9244477Z [ERROR]   Run 2: EmailOTPTestCase.testEmailOTPAuthentication:196 Error in redirection to email OTP authentication page for user: testuser2@wso2.com expected:<true> but was:<false>
2024-11-21T15:55:12.9246303Z [INFO] 
2024-11-21T15:55:12.9247125Z [ERROR]   LiteUserRegisterTestCase.testLiteUserRegistration:80 Lite user registration unsuccessful expected:<201> but was:<500>
2024-11-21T15:55:12.9248051Z [INFO] 

@darshanasbg darshanasbg changed the title Update framework, notification template versions and add default unicode support config Add capability to switch between storing notification content as binary data or text data Nov 22, 2024
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.

3 participants