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

Installing TranslatePress Pro on Learn #2445

Open
bsanevans opened this issue May 16, 2024 · 5 comments
Open

Installing TranslatePress Pro on Learn #2445

bsanevans opened this issue May 16, 2024 · 5 comments
Assignees
Labels
[Component] Environment Website development issues related to Learn's local environment, or needs Meta team involvement.

Comments

@bsanevans
Copy link
Contributor

bsanevans commented May 16, 2024

@sareiodata Dev work has started to launch Learning Pathways on Learn (project details) and a staging site has been created for that work: https://learn.wordpress.org/test/

TranslatePress has been installed and activated on this staging site so that we can test this out at the same time before putting it on the live site.

  1. Could we ask you for a perpetual license, as discussed in Test TranslatePress #2339 (comment)?
  2. Upon activating the plugin, @adamwoodnz is noticing a white-screen-of-death intermittently. This seems to be related to a missing TranslatePress table wp_trp_gettext_original_strings. Would you have any information about why this might be happening?
[15-May-2024 08:11:54 UTC] WordPress database error Table 'wp_trp_gettext_original_strings' doesn't exist for query SELECT id, original, domain, context FROM `wp_trp_gettext_original_strings` WHERE BINARY wp_trp_gettext_original_strings.original IN ('Manage Comments') made by TRP_Gettext_Insert_Update::gettext_original_strings_sync
@bsanevans bsanevans added the [Component] Environment Website development issues related to Learn's local environment, or needs Meta team involvement. label May 16, 2024
@bsanevans bsanevans self-assigned this May 16, 2024
@dd32
Copy link
Member

dd32 commented May 16, 2024

It appears that in addition to the license, we'll need to install the non-free plugin, so we'll need somewhere to download that from ongoing (as we can't use the WP plugin update process)

Additionally; Where should we file bugs for TranslatePress?

In addition to the WSOD reported above, where TP is failing to use the proper table prefix, it's also seemingly assuming it's active on other sites in the network, and running operations in the context of other blogs (ie. It's active on Blog ID 5, but during the admin toolbar switch_to_blog( 1 ) happens in the My sites list, the plugin then starts erroring because it's not active on ID 1, and wp_1_trp... doesn't exist, so it inserts logs in the error option, and that starts growing massive.. This is hard to explain.. :)

@sareiodata
Copy link

@bsanevans For license & paid addon please let me know of an email address to create an account and the license.
I can also create one on my own account and just provide the license and zip file. Regardless it works for me.

I'll get back to you regarding the tables not being created and the weird Multisite functionality. It's quite possible there are some bugs there, TP rarely gets used in Multisite.

As for bug reports, either the free WordPress forum or directly on our website.

@dd32
Copy link
Member

dd32 commented May 16, 2024

@sareiodata I think we can use learn @wordpress.org for the account.

@sareiodata
Copy link

Regarding the bugs:

  • we have an old ticket regarding switch_to_blog makes gettext insert repeated translations in a different subsite. So far we're not sure what's causing it, but will look at it next week.
  • as for the table, we never could replicate this issue. Is there a chance for an export or staging where this is happening?

Regarding the license:

@razvan-translatepress
Copy link

razvan-translatepress commented May 23, 2024

Hi,

I've managed to replicate the reported bug causing those SQL errors due to targeting the wrong site tables. I am currently working on a solution and I will get back to you with updates.

Thank you for providing lots of details about this issue.

Edit 29th May: It's still work in progress, we are trying to figure out a way to use the current site prefix instead of accidentally using the main site in specific functions related to the admin bar. This is due to the fact that $wpdb global changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Component] Environment Website development issues related to Learn's local environment, or needs Meta team involvement.
Projects
Status: In discussion
Development

No branches or pull requests

4 participants