Skip to content

[hi] fix: Add missing Honeypot#513

Merged
paulbalandan merged 1 commit intocodeigniter4:developfrom
patel-vansh:fix/hi-honeypot
Dec 12, 2025
Merged

[hi] fix: Add missing Honeypot#513
paulbalandan merged 1 commit intocodeigniter4:developfrom
patel-vansh:fix/hi-honeypot

Conversation

@patel-vansh
Copy link
Contributor

Description
Add missing Honeypot

Checklist:

  • Securely signed commits
  • Component(s) with PHPDoc blocks, only if necessary or adds value
  • Unit testing, with >80% coverage
  • User guide updated
  • Conforms to style guide

Copilot AI review requested due to automatic review settings December 11, 2025 04:25
Copy link

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 adds the missing Hindi (hi) translation file for Honeypot functionality and fixes a documentation issue in the Router.php file where a backslash character needed proper escaping in the English comment.

Key changes:

  • Added complete Honeypot.php language file for Hindi with all three required message keys
  • Corrected the backslash escape sequence in Router.php English comment from (\) to (\\)

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
Language/hi/Honeypot.php Adds new Hindi translation file for Honeypot with three message keys: noTemplate, noNameField, and theClientIsABot
Language/hi/Router.php Fixes the English comment to properly escape the backslash character in the invalidControllerName message

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@patel-vansh
Copy link
Contributor Author

Sorry for this copilot auto review stuff. I forgot to disable that setting. :)

@paulbalandan paulbalandan merged commit 9f7888c into codeigniter4:develop Dec 12, 2025
17 checks passed
@patel-vansh patel-vansh deleted the fix/hi-honeypot branch December 12, 2025 12:06
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