Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed load textdomain issue with WP 6.7 #4325

Merged
merged 6 commits into from
Nov 21, 2024
Merged

Conversation

girishpanchal30
Copy link
Contributor

Summary

I have removed the early translation function call following WordPress standards.

Check before Pull Request is ready:

Closes #4322

@girishpanchal30 girishpanchal30 added the pr-checklist-skip Allow this Pull Request to skip checklist. label Nov 19, 2024
@pirate-bot pirate-bot added pr-checklist-complete The Pull Request checklist is complete. (automatic label) labels Nov 19, 2024
@pirate-bot
Copy link
Collaborator

pirate-bot commented Nov 19, 2024

Plugin build for 3d5670c is ready 🛎️!

@rodica-andronache
Copy link
Contributor

tested and it's working well

@selul
Copy link
Contributor

selul commented Nov 20, 2024

@rodica-andronache @vytisbulkevicius is the whitelabel still working after this? Whitelabel for theme details.

@vytisbulkevicius
Copy link
Contributor

It still works:
image
image

But we need to apply same fix for https://github.com/Codeinwp/neve-pro-addon/ and check again then as there is still a warning coming from plugin code. @girishpanchal30, please apply the fix for neve-pro-addon as well

@girishpanchal30
Copy link
Contributor Author

@vytisbulkevicius Can you please share your test instance details?

@vytisbulkevicius
Copy link
Contributor

@girishpanchal30,

Sure:

Admin area URL: https://partcent.s1-tastewp.com/wp-admin
Username: bvytis
Password: tIFXkDiUUHw

@girishpanchal30
Copy link
Contributor Author

@selul @vytisbulkevicius I've reviewed the neve-pro-addon code and noticed that this function is being loaded too early, along with some translatable strings.

I attempted to move the code to the init hook, but doing so appears to break the WP_CLI commands for Neve theme.

I've been working on finding a solution for this issue. If you have any ideas or suggestions, please let me know.

Thanks!

@girishpanchal30
Copy link
Contributor Author

@selul @vytisbulkevicius I’ve loaded the CLI module separately with this PR, and it seems to be working fine.

Could you please test the WP-CLI commands registered by Neve and let me know if you encounter any issues?

Thanks!

@girishpanchal30
Copy link
Contributor Author

@selul @vytisbulkevicius Please ignore the neve-pro-addon PR. I’ve resolved the issue within the neve theme PR itself, and everything now appears to be working fine with neve-pro-addon.

@vytisbulkevicius
Copy link
Contributor

Tested and don't see warnings anymore. The Whitelabel still works so looks good.

@vytisbulkevicius vytisbulkevicius merged commit 5b524af into development Nov 21, 2024
15 checks passed
@vytisbulkevicius vytisbulkevicius deleted the bugfix/4322 branch November 21, 2024 18:24
@pirate-bot
Copy link
Collaborator

🎉 This PR is included in version 3.8.14 🎉

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 Nov 21, 2024
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.

_load_textdomain_just_in_time for translations / i18n loaded too early
6 participants