Skip to content

Fix/typo delet to delete#216

Closed
adikulkarni006 wants to merge 4 commits into
AnthropicBots:mainfrom
adikulkarni006:fix/typo-delet-to-delete
Closed

Fix/typo delet to delete#216
adikulkarni006 wants to merge 4 commits into
AnthropicBots:mainfrom
adikulkarni006:fix/typo-delet-to-delete

Conversation

@adikulkarni006

Copy link
Copy Markdown
Contributor

Closes #178

🐛 Problem

Admin panel had a spelling mistake – "Delet Product" instead of "Delete Product".

🔧 Fix

Changed Delet to Delete in the admin panel.

📁 Files Changed

  • frontend typo was there

✅ Testing

  • Verified button shows correct spelling "Delete".
  • Admin panel works as expected.

Ready for review.

@vercel

vercel Bot commented Jun 19, 2026

Copy link
Copy Markdown

@adikulkarni006 is attempting to deploy a commit to the Bhuvansh's projects Team on Vercel.

A member of the Team first needs to authorize it.

@BHUVANSH855

Copy link
Copy Markdown
Member

Kindly solve merge time conflicts.

@BHUVANSH855 BHUVANSH855 added action: clean-up Pull Request needs cleaning. SSoC26 Program label for Social Summer of Code Season 5. Merge Conflicts. PR has merge time conflicts, update your local branch and then re commit with updated changes. labels Jun 20, 2026
@BHUVANSH855 BHUVANSH855 removed the Merge Conflicts. PR has merge time conflicts, update your local branch and then re commit with updated changes. label Jun 20, 2026

@BHUVANSH855 BHUVANSH855 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for the contribution. I found a few issues that should be addressed before merging:

  • frontend/styles/admin.css: .delete-btn was changed to .Deletee-btn, but the existing markup and JS still use .delete-btn, so the delete button styling will not apply.
  • frontend/styles/style.css: the file appears truncated and removes existing search/autocomplete styles. Please verify that no existing styles were accidentally deleted.
  • frontend/scripts/auth.js and frontend/signup.html: the password strength meter introduces new behavior and uses inline styles. Please move styling to CSS and verify the signup flow still behaves as intended.

Please address these issues and request another review.

@BHUVANSH855 BHUVANSH855 added the Merge Conflicts. PR has merge time conflicts, update your local branch and then re commit with updated changes. label Jun 21, 2026
@BHUVANSH855

Copy link
Copy Markdown
Member

Kindly solve merge time commits.
I strongly advise you to make different branches for different issues.

@BHUVANSH855

Copy link
Copy Markdown
Member

@adikulkarni006
Have a look at the changes u made in the PR once.
Can u tell me is their was typo error earlier or now u created that.

And we need reason why u changed
Delete -> Deletee
?
Requesting review form @mohityadav8
Have a look and if you feel the changes are irrelevant, feel free to close this PR.

@BHUVANSH855 BHUVANSH855 added action: review Pull Request needs review. and removed action: clean-up Pull Request needs cleaning. Merge Conflicts. PR has merge time conflicts, update your local branch and then re commit with updated changes. labels Jun 23, 2026
@BHUVANSH855 BHUVANSH855 requested review from mohityadav8 and removed request for BHUVANSH855 June 23, 2026 03:58
@BHUVANSH855 BHUVANSH855 added the Merge Conflicts. PR has merge time conflicts, update your local branch and then re commit with updated changes. label Jun 23, 2026
@BHUVANSH855

Copy link
Copy Markdown
Member

Closing this PR, due to irrelevant changes, inactivity and merge conflicts.
You can file a new PR if interested to work on same issue.

Thanks

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

Labels

action: review Pull Request needs review. Merge Conflicts. PR has merge time conflicts, update your local branch and then re commit with updated changes. SSoC26 Program label for Social Summer of Code Season 5.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Admin dashboard button misspelled: "Delet" → "Delete"

2 participants