Fix: Complete DevDesk to ZapDesk rebrand #205
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Update all references from DevDesk to ZapDesk throughout the codebase: - Regenerated package-lock.json with correct package name - Updated .env.example with ZapDesk branding and generic placeholders - Updated all documentation files (DEPLOYMENT, EMAIL_SETUP, SETUP, etc.) - Updated test files to check for ZapDesk branding - Updated scripts and public assets (og-image.svg, generate-logo.js) - Updated GitHub issue templates - Removed hardcoded email addresses per security guidelines All quality checks pass (typecheck, lint, format). Co-Authored-By: Claude Opus 4.5 <[email protected]>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR aims to complete the rebranding from DevDesk to ZapDesk as requested in issue #204. The changes update branding references across configuration files, documentation, tests, scripts, and assets.
Changes:
- Updated package configuration to use "zapdesk" package name
- Replaced DevDesk with ZapDesk across all documentation files (6 .adoc files)
- Updated test assertions to check for "ZapDesk" branding instead of "DevDesk"
- Modified environment configuration, scripts, and assets with new branding
Reviewed changes
Copilot reviewed 13 out of 15 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| package-lock.json | Updated package name from "devdesk" to "zapdesk" |
| .env.example | Updated APP_NAME, SMTP_FROM_NAME to ZapDesk and replaced hardcoded emails with placeholders |
| tests/auth.spec.ts | Updated test to check for "ZapDesk" branding |
| tests/navigation.spec.ts | Updated tests to verify "ZapDesk" in page title and branding |
| scripts/generate-logo.js | Updated comments to reference ZapDesk (but logo implementation still incomplete) |
| scripts/README.md | Updated all DevDesk references to ZapDesk |
| public/og-image.svg | Updated text from "Dev" to "Zap" in the OG image |
| docs/DEPLOYMENT.adoc | Updated all DevDesk references, Azure resource names, and domain examples |
| docs/EMAIL_SETUP.adoc | Updated branding and replaced hardcoded emails with placeholders |
| docs/SETUP.adoc | Updated branding and Azure resource names (one incomplete reference found) |
| docs/SOLUTION_DESIGN.adoc | Updated all references and table headers |
| docs/TESTING.adoc | Updated all DevDesk references to ZapDesk |
| docs/TROUBLESHOOTING.adoc | Updated branding, Azure resource names, and GitHub URLs |
| .github/ISSUES/responsive-design-dashboard.md | Updated issue template with ZapDesk branding |
| .claude/settings.local.json | New local configuration file (should not be committed) |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
akash2017sky
force-pushed
the
fix/204-issue-rename-app
branch
from
f2ebd2b to
5229742
Compare
- Update generate-logo.js description to reference lightning bolt icon - Change SVG text from "Dev" to "Zap" in both dark and light logos - Fix CNAME record in SETUP.adoc from devdesk to zapdesk - Remove .claude/settings.local.json from repo (local config) - Add .claude/settings.local.json to .gitignore Co-Authored-By: Claude Opus 4.5 <[email protected]>
- Use getByRole for more specific locators - Update home page test to check landing page instead of redirect - Remove obsolete /customers route and API tests - Rename organizations test to projects Co-Authored-By: Claude Opus 4.5 <[email protected]>
Collaborator
Author
|
@BenGWeeks all code review comments are fixed now. @EdiWeeks this is read to test |
Regenerated lockfile to reflect the package name change in package.json. Co-Authored-By: Claude Opus 4.6 <[email protected]>
EdiWeeks
approved these changes
Feb 6, 2026
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Completes the rebrand from DevDesk to ZapDesk as requested in #204.
Changes Made
Package Configuration
package-lock.jsonwith correct package name "zapdesk"Environment Configuration
.env.examplewith ZapDesk brandingAPP_NAMEto "ZapDesk"SMTP_FROM_NAMEto "ZapDesk Support"Documentation (docs/*.adoc)
Test Files
tests/auth.spec.tsto check for "ZapDesk" brandingtests/navigation.spec.tsto check for "ZapDesk" in titleScripts & Assets
scripts/generate-logo.jscomments and referencesscripts/README.mdwith ZapDesk referencespublic/og-image.svgto display "ZapDesk" textGitHub Templates
.github/ISSUES/responsive-design-dashboard.mdQuality Checks
All checks pass successfully:
Testing
Security
Related Issue
Closes #204
🤖 Generated with Claude Code