Skip to content

Remove extensions navigation tip#5937

Merged
witoszekdev merged 4 commits intomainfrom
remove-old-navigation-link
Oct 6, 2025
Merged

Remove extensions navigation tip#5937
witoszekdev merged 4 commits intomainfrom
remove-old-navigation-link

Conversation

@witoszekdev
Copy link
Member

This PR removes message in configuration page for moved confgiuration to Extensions page.

Copilot AI review requested due to automatic review settings October 2, 2025 17:15
@witoszekdev witoszekdev requested a review from a team as a code owner October 2, 2025 17:15
@vercel
Copy link

vercel bot commented Oct 2, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Preview Comments Updated (UTC)
saleor-dashboard-storybook Ignored Ignored Preview Oct 6, 2025 5:49am

@changeset-bot
Copy link

changeset-bot bot commented Oct 2, 2025

🦋 Changeset detected

Latest commit: 444fcd4

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
saleor-dashboard Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes a navigation tip card that informed users about the relocation of Plugins and Webhook Events from the configuration page to the Extensions page. The change cleans up the configuration page by removing the temporary notification message and its associated imports.

  • Removed navigation tip card component and its content
  • Cleaned up unused imports (DashboardCard, ExtensionsUrls, useNavigator, ExclamationIcon, Button, FormattedMessage)
  • Removed the goToExtensions navigation function

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@codecov
Copy link

codecov bot commented Oct 2, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 40.06%. Comparing base (9acc950) to head (444fcd4).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5937      +/-   ##
==========================================
- Coverage   40.07%   40.06%   -0.01%     
==========================================
  Files        2473     2473              
  Lines       40190    40183       -7     
  Branches     8840     8840              
==========================================
- Hits        16107    16101       -6     
+ Misses      24056    24055       -1     
  Partials       27       27              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 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.

kzuraw
kzuraw previously approved these changes Oct 3, 2025
lkostrowski
lkostrowski previously approved these changes Oct 3, 2025
Copilot AI review requested due to automatic review settings October 3, 2025 11:57
@witoszekdev witoszekdev dismissed stale reviews from kzuraw and lkostrowski via 0d92e8b October 3, 2025 11:57
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@witoszekdev witoszekdev enabled auto-merge (squash) October 6, 2025 09:25
@witoszekdev witoszekdev merged commit bd3d8e3 into main Oct 6, 2025
16 checks passed
@witoszekdev witoszekdev deleted the remove-old-navigation-link branch October 6, 2025 09:25
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.

4 participants