Skip to content

Fix naming inconsistency: replace "PushiIt" with "PushIt" throughout codebase#5

Merged
skerbis merged 3 commits into
mainfrom
copilot/fix-4
Sep 6, 2025
Merged

Fix naming inconsistency: replace "PushiIt" with "PushIt" throughout codebase#5
skerbis merged 3 commits into
mainfrom
copilot/fix-4

Conversation

Copilot AI commented Sep 6, 2025

Copy link
Copy Markdown
Contributor

This PR standardizes the naming throughout the codebase by replacing all instances of "PushiIt" with the correct "PushIt" spelling. The addon is called "Push It" but was inconsistently referenced as "PushiIt" in many places.

Changes Made

JavaScript Files:

  • assets/backend.js: Updated all variable names, function names, and comments

    • window.PushiItwindow.PushIt
    • PushiItPublicKeyPushItPublicKey
    • waitForPushiIt()waitForPushIt()
    • checkPushiIt()checkPushIt()
    • PushiItBackendPushItBackend
    • PushiItResetPushItReset
  • assets/frontend.js: Updated API object and variable references

    • window.PushiItwindow.PushIt (main API object)
    • PushiItPublicKeyPushItPublicKey
    • PushiItUserIdPushItUserId
    • PushiItTopicsPushItTopics

PHP Pages:

  • pages/send.php: Fixed comments, JavaScript variables, and test message text
  • pages/settings.php: Updated frontend integration code snippet
  • pages/backend_notify.php: Fixed JavaScript variable names and function calls
  • pages/index.php: Corrected page title from "Pushi It" to "Push It"

Documentation:

  • README.md: Updated extension point examples from PUSHI_IT_* to PUSH_IT_*

Bug Fix:

  • install.php: Fixed syntax error (removed stray "/") and updated table references

All changes ensure consistent naming throughout the codebase. The JavaScript API remains functionally identical, just with corrected naming.

Fixes #4.


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

@skerbis

skerbis commented Sep 6, 2025

Copy link
Copy Markdown
Member

... while maintaining backward compatibility ..is not needed because still in development @copilot

Copilot AI and others added 2 commits September 6, 2025 15:47
…nstall.php

Co-authored-by: skerbis <791247+skerbis@users.noreply.github.com>
…PHP pages and README

Co-authored-by: skerbis <791247+skerbis@users.noreply.github.com>
Copilot AI changed the title [WIP] PushiIt ersetzen durch Pushit Fix naming inconsistency: replace "PushiIt" with "PushIt" throughout codebase Sep 6, 2025
Copilot AI requested a review from skerbis September 6, 2025 15:51

Copilot AI commented Sep 6, 2025

Copy link
Copy Markdown
Contributor Author

... while maintaining backward compatibility ..is not needed because still in development @copilot

You're absolutely right! Since the addon is still in development, backward compatibility concerns don't apply. The changes can be made without those considerations. Thanks for pointing that out.

@skerbis skerbis marked this pull request as ready for review September 6, 2025 15:55
@skerbis skerbis merged commit 288e6e4 into main Sep 6, 2025
1 check passed
@skerbis skerbis deleted the copilot/fix-4 branch September 6, 2025 16:13
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.

PushiIt ersetzen durch Pushit

2 participants