Skip to content

Remove demo mode code#5881

Merged
lkostrowski merged 3 commits intomainfrom
remove-demo-mode
Sep 22, 2025
Merged

Remove demo mode code#5881
lkostrowski merged 3 commits intomainfrom
remove-demo-mode

Conversation

@kzuraw
Copy link
Contributor

@kzuraw kzuraw commented Sep 18, 2025

Scope of the change

Copilot AI review requested due to automatic review settings September 18, 2025 13:42
@changeset-bot
Copy link

changeset-bot bot commented Sep 18, 2025

🦋 Changeset detected

Latest commit: 163171d

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

@vercel
Copy link

vercel bot commented Sep 18, 2025

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

Project Deployment Preview Comments Updated (UTC)
saleor-dashboard-storybook Ready Ready Preview Comment Sep 22, 2025 7:17am

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 demo mode functionality from the Saleor Dashboard application. The changes eliminate all code related to the demo mode feature, including configuration, components, and conditional logic.

  • Removes DEMO_MODE environment variable and configuration
  • Deletes the DemoBanner component and related files
  • Removes demo-specific login form pre-population and notifications
  • Cleans up conditional demo mode logic throughout the application

Reviewed Changes

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

Show a summary per file
File Description
vite.config.js Removes DEMO_MODE from environment variables and OG tags injection
src/intl.ts Removes demo mode notification message definition
src/index.tsx Removes DemoBanner component import and usage
src/index.html Removes OG tags injection placeholder
src/config.ts Removes DEMO_MODE configuration export
src/components/DemoBanner/* Deletes entire DemoBanner component directory
src/auth/utils.ts Removes displayDemoMessage function
src/auth/hooks/useAuthProvider.ts Removes demo mode notifications and simplifies external login logic
src/auth/components/LoginPage/form.tsx Removes demo mode login form pre-population
.storybook/vite.storybook.config.js Removes DEMO_MODE from Storybook configuration
.env.template Removes DEMO_MODE environment variable
.changeset/cold-cooks-shout.md Documents the removal of demo mode code

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

@github-actions github-actions bot temporarily deployed to pr-5881 September 18, 2025 13:45 Destroyed
@kzuraw kzuraw marked this pull request as ready for review September 18, 2025 13:52
@kzuraw kzuraw requested a review from a team as a code owner September 18, 2025 13:52
@codecov
Copy link

codecov bot commented Sep 18, 2025

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 38.64%. Comparing base (a2c1987) to head (163171d).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
src/auth/hooks/useAuthProvider.ts 0.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##             main    #5881    +/-   ##
========================================
  Coverage   38.64%   38.64%            
========================================
  Files        2745     2742     -3     
  Lines       43254    43228    -26     
  Branches     9455     9801   +346     
========================================
- Hits        16714    16707     -7     
+ Misses      26510    26491    -19     
  Partials       30       30            

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

witoszekdev
witoszekdev previously approved these changes Sep 18, 2025
lkostrowski
lkostrowski previously approved these changes Sep 22, 2025
Copilot AI review requested due to automatic review settings September 22, 2025 07:14
@lkostrowski lkostrowski dismissed stale reviews from witoszekdev and themself via 163171d September 22, 2025 07:14
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 16 out of 16 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.

@lkostrowski lkostrowski enabled auto-merge (squash) September 22, 2025 07:18
@lkostrowski lkostrowski merged commit cee2ff1 into main Sep 22, 2025
14 of 15 checks passed
@lkostrowski lkostrowski deleted the remove-demo-mode branch September 22, 2025 07:18
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