Skip to content

Remove duplicate Toaster component from root layout#612

Merged
jthrilly merged 2 commits intonextfrom
copilot/sub-pr-610-again
Feb 9, 2026
Merged

Remove duplicate Toaster component from root layout#612
jthrilly merged 2 commits intonextfrom
copilot/sub-pr-610-again

Conversation

Copy link
Contributor

Copilot AI commented Feb 9, 2026

The <Toaster /> component was being rendered twice: once in app/layout.tsx and once in components/Providers/index.tsx, creating duplicate toast containers and announcements.

Changes:

  • Removed <Toaster /> from app/layout.tsx (line 24)
  • Removed unused Toaster import from app/layout.tsx

The component now renders only once through the centralized Providers wrapper, as intended.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI mentioned this pull request Feb 9, 2026
6 tasks
@vercel
Copy link

vercel bot commented Feb 9, 2026

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

Project Deployment Actions Updated (UTC)
fresco-sandbox Ready Ready Preview, Comment Feb 9, 2026 11:42am
fresco-storybook Canceled Canceled Feb 9, 2026 11:42am

Request Review

Co-authored-by: jthrilly <1387940+jthrilly@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Feb 9, 2026

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • checkpoint.prisma.io
    • Triggering command: /opt/hostedtoolcache/node/24.13.0/x64/bin/node /opt/hostedtoolcache/node/24.13.0/x64/bin/node /home/REDACTED/work/Fresco/Fresco/node_modules/.pnpm/prisma@7.3.0_@types&#43;react@18.3.18_react-dom@18.3.1_react@18.3.1__react@18.3.1_typescript@5.9.3/node_modules/prisma/build/child {&#34;product&#34;:&#34;prisma&#34;,&#34;version&#34;:&#34;7.3.0&#34;,&#34;cli_install_type&#34;:&#34;local&#34;,&#34;information&#34;:&#34;&#34;,&#34;local_timestamp&#34;:&#34;2026-02-09T11:36:29Z&#34;,&#34;project_hash&#34;:&#34;ae2c8f24&#34;,&#34;cli_path&#34;:&#34;/home/REDACTED/work/Fresco/Fresco/node_modules/prisma/build/index.js&#34;,&#34;cli_path_hash&#34;:&#34;fb3d3539&#34; (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title [WIP] Address feedback from pull request discussion for v4.0.0 updates Remove duplicate Toaster component from root layout Feb 9, 2026
Copilot AI requested a review from jthrilly February 9, 2026 11:38
@jthrilly jthrilly marked this pull request as ready for review February 9, 2026 11:38
Copilot AI review requested due to automatic review settings February 9, 2026 11:39
@jthrilly jthrilly merged commit eae2470 into next Feb 9, 2026
4 of 6 checks passed
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

Removes the duplicate global toast container by relying on the centralized <Providers> wrapper to render <Toaster />, rather than rendering it both in the root layout and in providers.

Changes:

  • Removed the <Toaster /> render from app/layout.tsx.
  • Removed the now-unused Toaster import from app/layout.tsx.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

3 participants