Skip to content

Handled undefined property to prevent error#4544

Merged
selul merged 2 commits into
developmentfrom
bugfix/pro/3194
Jul 21, 2026
Merged

Handled undefined property to prevent error#4544
selul merged 2 commits into
developmentfrom
bugfix/pro/3194

Conversation

@girishpanchal30

Copy link
Copy Markdown
Contributor

Check before Pull Request is ready:

Closes https://github.com/Codeinwp/neve-pro-addon/issues/3194

@girishpanchal30
girishpanchal30 requested a review from Copilot July 17, 2026 12:00
@girishpanchal30 girishpanchal30 added the pr-checklist-skip Allow this Pull Request to skip checklist. label Jul 17, 2026
@pirate-bot pirate-bot added the pr-checklist-complete The Pull Request checklist is complete. (automatic label) label Jul 17, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Prevents frontend errors when the NeveProperties global is unavailable.

Changes:

  • Adds safe existence checks for mobile menu behavior.
  • Adds an infinite-scroll guard, though it occurs after an earlier unsafe access.

Reviewed changes

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

File Description
assets/js/src/frontend/hgf.js Safely handles missing localization data in menu toggling.
assets/js/src/frontend/blog.js Guards infinite scroll against missing localization data.

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

Comment thread assets/js/src/frontend/blog.js Outdated
@pirate-bot

pirate-bot commented Jul 17, 2026

Copy link
Copy Markdown
Collaborator

Plugin build for 1287244 is ready 🛎️!

@selul
selul merged commit 853fd9e into development Jul 21, 2026
14 checks passed
@selul
selul deleted the bugfix/pro/3194 branch July 21, 2026 13:58
@pirate-bot

Copy link
Copy Markdown
Collaborator

🎉 This PR is included in version 4.2.9 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@pirate-bot pirate-bot added the released Indicate that an issue has been resolved and released in a particular version of the product. label Jul 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-checklist-complete The Pull Request checklist is complete. (automatic label) pr-checklist-skip Allow this Pull Request to skip checklist. released Indicate that an issue has been resolved and released in a particular version of the product.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants