Skip to content

fix: remove migration banner#175

Merged
fmorency merged 1 commit intoliftedinit:mainfrom
fmorency:no-migration-banner
Jun 10, 2025
Merged

fix: remove migration banner#175
fmorency merged 1 commit intoliftedinit:mainfrom
fmorency:no-migration-banner

Conversation

@fmorency
Copy link
Copy Markdown
Contributor

This pull request includes a small change to the src/views/layout/app-nav.tsx file. The change comments out the MigrationBanner component, effectively removing it from the application navigation layout.

@fmorency fmorency requested a review from Copilot June 10, 2025 17:42
@fmorency fmorency self-assigned this Jun 10, 2025
@netlify
Copy link
Copy Markdown

netlify Bot commented Jun 10, 2025

Deploy Preview for lifted-alberto ready!

Name Link
🔨 Latest commit b018eb7
🔍 Latest deploy log https://app.netlify.com/projects/lifted-alberto/deploys/68486e7eb667720008f6be4b
😎 Deploy Preview https://deploy-preview-175--lifted-alberto.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Copy Markdown

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 the MigrationBanner component from the application navigation layout by commenting out both its import and usage.

  • Removes the MigrationBanner import.
  • Comments out the MigrationBanner component in the JSX.

@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 10, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 45.67%. Comparing base (453c3d6) to head (b018eb7).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #175      +/-   ##
==========================================
+ Coverage   45.61%   45.67%   +0.06%     
==========================================
  Files         122      122              
  Lines        3229     3229              
  Branches      897      897              
==========================================
+ Hits         1473     1475       +2     
+ Misses       1741     1740       -1     
+ Partials       15       14       -1     

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

@fmorency fmorency merged commit 0ccc902 into liftedinit:main Jun 10, 2025
10 checks passed
@fmorency fmorency deleted the no-migration-banner branch June 10, 2025 17:51
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.

2 participants