Skip to content

[WIP] Remove event publishing UI#10526

Draft
pavinduLakshan wants to merge 1 commit into
wso2:masterfrom
pavinduLakshan:remove_events_feature
Draft

[WIP] Remove event publishing UI#10526
pavinduLakshan wants to merge 1 commit into
wso2:masterfrom
pavinduLakshan:remove_events_feature

Conversation

@pavinduLakshan

Copy link
Copy Markdown
Member

Purpose

$subject

Related Issues

  • N/A

Related PRs

  • N/A

Checklist

  • e2e cypress tests locally verified. (for internal contributers)
  • Manual test round performed and verified.
  • UX/UI review done on the final implementation.
  • Documentation provided. (Add links if there are any)
  • Relevant backend changes deployed and verified
  • Unit tests provided. (Add links if there are any)
  • Integration tests provided. (Add links if there are any)

Security checks

Developer Checklist (Mandatory)

  • Complete the Developer Checklist in the related product-is issue to track any behavioral change or migration impact.

@pavinduLakshan
pavinduLakshan marked this pull request as draft July 11, 2026 12:05
@coderabbitai

coderabbitai Bot commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: d0962c71-b7a8-475d-b37a-a7ed870c1494

📥 Commits

Reviewing files that changed from the base of the PR and between ccdfbc9 and 80ca359.

📒 Files selected for processing (7)
  • apps/console/java/org.wso2.identity.apps.console.server.feature/resources/deployment.config.json.j2
  • apps/console/src/public/deployment.config.json
  • features/admin.core.v1/constants/app-constants.ts
  • features/admin.core.v1/models/config.ts
  • features/admin.core.v1/utils/route-utils.ts
  • modules/access-control/src/constants/access-control-constants.ts
  • modules/core/src/helpers/__mocks__/deployment.config.json
💤 Files with no reviewable changes (6)
  • features/admin.core.v1/utils/route-utils.ts
  • modules/access-control/src/constants/access-control-constants.ts
  • modules/core/src/helpers/mocks/deployment.config.json
  • features/admin.core.v1/constants/app-constants.ts
  • features/admin.core.v1/models/config.ts
  • apps/console/java/org.wso2.identity.apps.console.server.feature/resources/deployment.config.json.j2

📝 Walkthrough

Walkthrough

Changes

Console feature configuration and legacy event removal

Layer / File(s) Summary
Console feature configuration
apps/console/src/public/deployment.config.json
Adds the enabled FAPI feature, updates flow extension scopes, removes the duplicate flow extension definition, and reorders application configuration entries.
Event route and access removal
features/admin.core.v1/..., modules/access-control/...
Removes the event feature flag, edit route, navigation mapping, and event permission constants.
Deployment configuration cleanup
apps/console/java/.../deployment.config.json.j2, modules/core/.../deployment.config.json
Removes event configuration generation from the console template and event feature data from the deployment mock.

Possibly related PRs


Important

Pre-merge checks failed

Please resolve all errors before merging. Addressing warnings is optional.

❌ Failed checks (1 error, 1 warning)

Check name Status Explanation Resolution
Changeset Required ❌ Error PR changes feature/module code but no .changeset/*.md file is listed among the changed files, so the required changeset is missing. Add a new .changeset/*.md file (not README.md) covering the affected packages and version bumps, especially the console/myaccount/core packages.
Description check ⚠️ Warning The description keeps the template but leaves the Purpose section as a placeholder and omits concrete implementation details. Replace placeholders with a real Purpose summary, add any related issue/PR links, and complete the relevant checklist items.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title is concise and accurately summarizes the main change: removing the event publishing UI.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • 🛠️ create changeset

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codecov

codecov Bot commented Jul 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.84%. Comparing base (b3e74d2) to head (80ca359).
⚠️ Report is 15 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #10526      +/-   ##
==========================================
+ Coverage   72.82%   72.84%   +0.01%     
==========================================
  Files         470      470              
  Lines       71162    71181      +19     
  Branches      240      240              
==========================================
+ Hits        51826    51849      +23     
+ Misses      19225    19221       -4     
  Partials      111      111              
Files with missing lines Coverage Δ
...-control/src/constants/access-control-constants.ts 1.56% <ø> (+0.09%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

1 participant