Skip to content

Seperator -> Separator#823

Merged
danditomaso merged 1 commit into
meshtastic:mainfrom
philon-:fix-rename-seperator
Sep 2, 2025
Merged

Seperator -> Separator#823
danditomaso merged 1 commit into
meshtastic:mainfrom
philon-:fix-rename-seperator

Conversation

@philon-

@philon- philon- commented Sep 1, 2025

Copy link
Copy Markdown
Contributor

Description

This PR changes the file name of the Separator component from Seperator.tsx to Separator.tsx

Related Issues

Changes Made

Testing Done

Screenshots (if applicable)

Checklist

  • Code follows project style guidelines
  • Documentation has been updated or added
  • Tests have been added or updated
  • All i18n translation labels have been added (read
    CONTRIBUTING_I18N_DEVELOPER_GUIDE.md for more details)

Copilot AI review requested due to automatic review settings September 1, 2025 15:17
@vercel

vercel Bot commented Sep 1, 2025

Copy link
Copy Markdown

@philon- is attempting to deploy a commit to the Meshtastic Team on Vercel.

A member of the Team first needs to authorize it.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 corrects a spelling error by renaming the Separator component file from Seperator.tsx to Separator.tsx and updates all import references accordingly.

  • Fixes the misspelled filename from "Seperator" to "Separator"
  • Updates import statements across multiple files to reference the corrected filename
  • Standardizes the import path format to use @app/components/UI/Separator

Reviewed Changes

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

Show a summary per file
File Description
packages/web/src/pages/Dashboard/index.tsx Updates import to use corrected Separator path
packages/web/src/components/UI/Separator.tsx Updates React ref type from ElementRef to ComponentRef
packages/web/src/components/PageComponents/Map/NodeDetail.tsx Updates import to use corrected Separator path
packages/web/src/components/Dialog/RebootDialog.tsx Updates import to use corrected Separator path
packages/web/src/components/Dialog/NodeDetailsDialog/NodeDetailsDialog.tsx Updates import to use corrected Separator path

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

Comment thread packages/web/src/components/UI/Separator.tsx
@philon- philon- force-pushed the fix-rename-seperator branch from bcd69bf to 67c0f80 Compare September 1, 2025 15:19
@vercel

vercel Bot commented Sep 2, 2025

Copy link
Copy Markdown

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

Project Deployment Preview Comments Updated (UTC)
web-test Ready Ready Preview Comment Sep 2, 2025 1:57am

@danditomaso danditomaso merged commit 3582499 into meshtastic:main Sep 2, 2025
4 checks passed
danditomaso added a commit that referenced this pull request Sep 2, 2025
danditomaso added a commit that referenced this pull request Sep 2, 2025
philon- added a commit to philon-/meshtastic-web that referenced this pull request Sep 2, 2025
@philon- philon- mentioned this pull request Sep 2, 2025
4 tasks
danditomaso added a commit that referenced this pull request Sep 3, 2025
* Reapply "Seperator -> Separator (#823)" (#827)

This reverts commit e421eb4.

* Rename new seperator

* Update packages/web/src/components/Dialog/NodeDetailsDialog/NodeDetailsDialog.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update packages/web/src/components/Dialog/RebootDialog.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update packages/web/src/components/PageComponents/Map/NodeDetail.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update packages/web/src/pages/Dashboard/index.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: philon- <philon-@users.noreply.github.com>
Co-authored-by: Dan Ditomaso <dan.ditomaso@gmail.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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