-
Notifications
You must be signed in to change notification settings - Fork 552
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 fallback to IDN_ACTION_PROPERTIES table #6578
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #6578 +/- ##
============================================
- Coverage 47.15% 47.12% -0.03%
+ Complexity 15561 15504 -57
============================================
Files 1779 1779
Lines 106913 107030 +117
Branches 20201 20222 +21
============================================
+ Hits 50411 50440 +29
- Misses 49427 49502 +75
- Partials 7075 7088 +13
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
...va/org/wso2/carbon/identity/action/management/internal/dao/impl/ActionManagementDAOImpl.java
Outdated
Show resolved
Hide resolved
...va/org/wso2/carbon/identity/action/management/internal/dao/impl/ActionManagementDAOImpl.java
Outdated
Show resolved
Hide resolved
...va/org/wso2/carbon/identity/action/management/internal/dao/impl/ActionManagementDAOImpl.java
Outdated
Show resolved
Hide resolved
PR builder started |
PR builder completed |
|
Integration test failures are intermittent and not relevant to the changes |
Proposed changes in this pull request
This PR adds the backward compatibilty in BE to support the future schema change to be done in IDN_ACTION_PROPERTIES table. When the PROPERTY_VALUE column is getting changed to PRIMITIVE_VALUE column this BE logic handles the CRUD operations.
When should this PR be merged
Soon
Follow up actions
Related issue