Skip to content

Conversation

@felicio
Copy link
Collaborator

@felicio felicio commented Nov 18, 2025

why

Do we still use E: labels? – Igor
...
I started removing the old ones in Desktop – Jo
...
Yeah, let's kill those labels – Volo

Copilot AI review requested due to automatic review settings November 18, 2025 14:25
@changeset-bot
Copy link

changeset-bot bot commented Nov 18, 2025

⚠️ No Changeset found

Latest commit: dca1d09

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

@vercel
Copy link

vercel bot commented Nov 18, 2025

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

Project Deployment Preview Comments Updated (UTC)
status-portfolio Ready Ready Preview Comment Nov 18, 2025 2:39pm
status-website Ready Ready Preview Comment Nov 18, 2025 2:39pm
4 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
status-api Ignored Ignored Preview Nov 18, 2025 2:39pm
status-components Ignored Ignored Preview Nov 18, 2025 2:39pm
status-network-hub Ignored Ignored Preview Nov 18, 2025 2:39pm
status-network-website Ignored Ignored Preview Nov 18, 2025 2:39pm

Copilot finished reviewing on behalf of felicio November 18, 2025 14:26
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 the admin insights functionality from the status.app website, including all related pages, components, and GraphQL dependencies.

Key changes:

  • Complete removal of the /insights route and all sub-routes (epics, orphans, repos)
  • Removal of GraphQL runtime dependencies (graphql, graphql-request) from production
  • Removal of GraphQL codegen dev dependencies (@graphql-codegen/* packages)
  • Package lock updates reflecting dependency changes and version bumps

Reviewed Changes

Copilot reviewed 44 out of 46 changed files in this pull request and generated 1 comment.

File Description
pnpm-lock.yaml Removed graphql and graphql-request from dependencies, removed @graphql-codegen packages from devDependencies, updated various package versions
apps/status.app/src/app/(website)/insights/** Deleted entire insights directory including layout, pages, and components

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

Comment on lines 1173 to +1178
graphql:
specifier: ^16.7.1
version: 16.11.0
version: 16.12.0
graphql-request:
specifier: ^6.1.0
version: 6.1.0(graphql@16.11.0)
version: 6.1.0(graphql@16.12.0)
Copy link

Copilot AI Nov 18, 2025

Choose a reason for hiding this comment

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

The graphql and graphql-request dependencies are still present in what appears to be another importer section (lines 1173-1178). This suggests these packages may still be needed elsewhere in the monorepo. Please verify that removing these from the main app dependencies doesn't break other packages that depend on them.

Copilot uses AI. Check for mistakes.
Copy link
Collaborator

@jkbktl jkbktl left a comment

Choose a reason for hiding this comment

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

2 participants