-
Notifications
You must be signed in to change notification settings - Fork 0
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
MLA Action Network site - header and footer issues #151
Comments
The header and footer are set in the Charity Hub theme options page. For some reason these options seem not to have been retained in the transfer from prod to dev. It could also be that a plugin update somehow cleared the site's options table? See #154 for similar issue. |
It could be. I'll do a sync tonight to helnlein. That might be a good place
to isolate this issue.
…On Thu, Oct 7, 2021 at 3:38 PM mikethicke ***@***.***> wrote:
The header and footer are set in the Charity Hub theme options page. For
some reason these options seem not to have been retained in the transfer
from prod to dev. It could also be that a plugin update somehow cleared the
site's options table?
See #154 <#154> for
similar issue.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#151 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AB7ZYVR2Z5KHMCXHNH2UL53UFXZMZANCNFSM5EWHOYIQ>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
--
R. Eric Knappe
Head of Software Development
Modern Language Association
85 Broad St. Suite 500
NY, NY 10004
(646) 576-5030
www.mla.org
|
I have completed the sync of heinlein (still running WP 4.9.6) First question: If, so I can do some upgrades (wp first) and check again. Please list sites checked for settings |
Action: the settings don't seem to be intact. I haven't located the footer setting yet, but social sharing options aren't retained; logo is missing. The footer is not displaying (https://action.mla.heinlein.mlacommons.org/wp-admin/admin.php?page=charityhub). On the site overall formatting from the theme seems to have been lost. On https://president.mla.heinlein.mlacommons.org/ and https://ask.up.heinlein.mlacommons.org/ (both on 15zine) settings seem to be retained (ticket #154). |
Eric, do you just use sync_db_and_files_from_hcommons.bash to do the sync? Is there anything more to it? |
Well I run it in steps. Why do you ask.
…On Mon, Oct 11, 2021 at 3:15 PM mikethicke ***@***.***> wrote:
Eric, do you just use sync_db_and_files_from_hcommons.bash to do the sync?
Is there anything more to it?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#151 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AB7ZYVR6XAVFB33REATQCHTUGMZTTANCNFSM5EWHOYIQ>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
--
R. Eric Knappe
Head of Software Development
Modern Language Association
85 Broad St. Suite 500
NY, NY 10004
(646) 576-5030
www.mla.org
|
Just trying to understand what is going on here. |
action.mla.hcommons.org charity hub theme options query. select option_value from wp_1000642_options where option_name='crth_admin_option'; The value is not what I expected in heinlein - string should not start with "a:328" since the string length is longer in heinlein. I'm using https://github.com/Automattic/go-search-replace because wp cli search replace no longer works with our size db. |
Automattic's own search replace script isn't doing serialization correctly? |
This theory appears to be incorrect, an option value containing a serialized array contains a number of counters. The first one "a:nnn" contains the number of array elements. Within the string are "s:" and "i:" elements, the value of an s: element contains text, for example hcommons.org in prod which is replaced with heinlein.mlacommons.org in this example. This will cause an increase in the s: value. This is the case with the charity hub admin option for action.mla.heinlein.mlacommons.org. |
That being said. The option value that controls charity_hub settings in heinlein appears to be corrupt. The following small program works in prod but fails in dev. ubuntu@heinlein:~/utils/check_option.php global $wpdb; |
I think the theory is correct in the end. Comparing the serialized values for crth_admin_option->additional-style, on hcommons the string length is reported as So it seems that the GO script is perhaps failing to do string serialization correctly for very long strings, or maybe there is some special character that it's choking on. |
I'm pretty sure this is a bug in the go-search-replace script. I submitted an issue here: Automattic/go-search-replace#33 |
Confirmed as a bug withe the go-search-replace script: Automattic/go-search-replace#33 (comment) |
Eric:
|
It turns out that CharityHub creates a custom stylesheet at I can't actually test this theory because I can't login to the dashboard on mla.heinlein.mlacommons.org because I get a SAML error about a missing employee number. |
This appears to be fixed with a successful sync. |
Tester (if different from issue creator):
Site: MLA Action Network
URL: https://action.mla.hcommons-dev.org/
Test Account:
Area (CORE, sites, profiles, groups, general): Sites
If issue discovered on dev, does it also exist on prod?
Describe the issue
The Action header and top horizontal menu items are missing; a theme header (charity hub because we care) appears instead.
The footer is also missing content.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Prod and dev should display identically.
Screenshots
If applicable, add screenshots to help explain your problem.
Optional details, if they seem relevant
Desktop (please complete the following information):
Smartphone (please complete the following information):
Additional context
Add any other context about the problem here.
The text was updated successfully, but these errors were encountered: