Skip to content

Conversation

@ericglau
Copy link
Member

Temporarily disables the Polkadot tab from the UI. Keeps underlying logic, since we'll plan to re-enable it perhaps with more context.

@ericglau ericglau requested a review from a team as a code owner September 29, 2025 16:06
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Sep 29, 2025

Walkthrough

Removed the Polkadot option from the UI by deleting the polkadot.html page and removing the Polkadot navigation link from Cairo, Index, Stellar, and Stylus public pages.

Changes

Cohort / File(s) Summary
Remove Polkadot nav link
packages/ui/public/cairo.html, packages/ui/public/index.html, packages/ui/public/stellar.html, packages/ui/public/stylus.html
Deleted the Polkadot navigation item from page headers; no other structural changes.
Delete Polkadot page
packages/ui/public/polkadot.html
Removed the entire Polkadot public page (static HTML with associated includes).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title Check ✅ Passed The title clearly identifies the removal of the Polkadot tab from the UI, which is the primary change introduced by this pull request. It is concise and directly references the main modification without extraneous details or vague language. A teammate glancing at the history can immediately understand that the Polkadot navigation option is being removed.
Description Check ✅ Passed The description accurately explains that the pull request temporarily disables the Polkadot tab while preserving the underlying logic for future re-enablement, which directly matches the changes in the PR. It is clearly related to the modifications and succinctly communicates the intent without drifting off-topic. This level of detail is sufficient for a lenient description check.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 450ab48 and 8801a79.

📒 Files selected for processing (5)
  • packages/ui/public/cairo.html (0 hunks)
  • packages/ui/public/index.html (0 hunks)
  • packages/ui/public/polkadot.html (0 hunks)
  • packages/ui/public/stellar.html (0 hunks)
  • packages/ui/public/stylus.html (0 hunks)
💤 Files with no reviewable changes (5)
  • packages/ui/public/stylus.html
  • packages/ui/public/stellar.html
  • packages/ui/public/index.html
  • packages/ui/public/cairo.html
  • packages/ui/public/polkadot.html
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: build (solidity, default)
  • GitHub Check: semgrep-cloud-platform/scan

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 and usage tips.

@ericglau ericglau enabled auto-merge (squash) September 29, 2025 16:09
@ericglau ericglau merged commit 6b31f47 into OpenZeppelin:master Sep 29, 2025
18 of 19 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Sep 29, 2025
@ericglau ericglau deleted the tempremovepolkadot branch September 29, 2025 17:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants