View Issue Details

IDProjectCategoryView StatusLast Update
5238Composr alpha bug reportsGeneral / Uncategorisedpublic2023-02-10 14:30
ReporterPDStig Assigned ToChris Graham  
PrioritynormalSeverityfeature 
Status resolvedResolutionfixed 
Summary5238: Social sharing: Need to test Facebook, Tumblr, VK
DescriptionFacebook, Tumblr, VK require an account to share something and do not offer any previews prior to login as to what the content would look like (thus I cannot verify that sharing works correctly). I do not have accounts on these platforms.
TagsNo tags attached.
Attach Tags
Sponsorship open

Sponsor

Date Added Member Amount Sponsored

Activities

Adam Edington

2023-01-13 14:08

administrator   ~7867

I can test VK and Facebook, also happy to create an account at Tumblr.

Chris Graham

2023-01-14 23:12

administrator   ~7869

Thanks Adam. Feel free to try installing v11 again from Git and see if it works!

Adam Edington

2023-01-15 11:56

administrator   ~7873

I tried a couple of days ago and managed to install on localhost, so that's good. However the menu modules and other key modules and blocks were claimed as missing so I couldn't really do much after that. Will try installing on my hosting account later this evening though.

Adam Edington

2023-01-17 00:25

administrator   ~7875

Stuck on Step 7 of installer:-
localhost gives no error just times out
live hosting gives the error "Parse error: syntax error, unexpected string content "docs/pages/comcode_cu", expecting "]" in sources_custom/hooks/systems/addon_registry/composr_tutorials.php on line 1577

Also of note, I use peazip and I have errors extracting the v11 tar or zip. Unspecified errors but thought I'd mention it.

Chris Graham

2023-01-17 02:34

administrator   ~7876

Patrick/Adam, could you connect on this, maybe set a time to do a screenshare session or something and hammer through whatever bugs come up?

Chris Graham

2023-01-17 03:02

administrator   ~7877

Last edited: 2023-01-17 03:03

That line "line 1577" sounds like maybe you extracted from a TAR file that had not fully downloaded.
EDIT: The issue about missing blocks and modules likely also would have been caused by an incomplete download.

The issue with Peazip is now fixed. There was a symlink in the Git repository, which didn't agree with Windows. I've removed the symlink and added it to our testing for that to not happen again.

Guest

2023-01-17 22:02

reporter   ~7878

Last edited: 2023-01-18 01:44

View 1 revisions

Still timing out on Step 7 after a long wait when step 6 completes on local, uploading to live... happy to be involved in a bug hunt but I'll let Patrick pick time and date.

Adam Edington

2023-01-18 02:48

administrator   ~7879

Installed on live, I did notice the installer put a port 80 addition despite not being on localhost. Onto the share testing...

Chris Graham

2023-01-18 03:17

administrator   ~7880

Can you clarify about port 80. Do you mean it's in your base URL? Maybe show a screenshot of what you mean.

As for installation timeouts, I've just done some benchmarking and optimisation but I can see more is required. Step 7 is 13 second son my machine, Step 8 is 20 seconds.

Adam Edington

2023-01-18 03:28

administrator   ~7881

Last edited: 2023-01-18 03:31

Sharing works to all 3 services and previews/end results are what would be expected, though it selects a random header banner as the post image so that will need defining (a default for text only content?). Tumblr lets me select a header banner or the advertise here banner ;)

Up and running v11, yay. Yes, Port 80 was added to the base URL by the installer on my live hosting but I noticed and removed it.

Chris Graham

2023-01-18 16:37

administrator   ~7882

Thanks Adam.

Just on the installer again...

I've optimised it to Step 7 as 10 seconds, and Step 8 as 12 seconds. So roughly 1/3 faster.
I'm not too worried about the installer performance from Git given that there is a lot more to install from Git due to all the non-bundled addons being included. But users may use some very slow servers so it was worth taking the chance to make the gains I reasonably could.

Adam Edington

2023-01-18 19:29

administrator   ~7883

Nice, will retry on wampstack this evening.

Chris Graham

2023-02-10 14:30

administrator   ~7898

Regarding social services using the header image...

There's some code in HTML_HEAD.tpl:
{+START,IF_EMPTY,{$METADATA,image}}
    {$SET,fallback_image,{$IMG,logo/standalone_logo}}
    <meta property="og:image" content="{$GET*,fallback_image}" />
    <meta property="og:image:width" content="{$IMG_WIDTH*,{$GET,fallback_image}}" />
    <meta property="og:image:height" content="{$IMG_HEIGHT*,{$GET,fallback_image}}" />
{+END}

It says to use the logo for the Open Graph image if there is no natural image on the page (natural image might be the images associated with the news article being viewed, for example).

The Facebook tutorial contains a section "Vastly simplifying the images", which shows how to make a new custom Open Graph image, should you want better than that. I'm renaming that tutorial "Composr Supplementary: Facebook Support / Open Graph" to indicate it also documents Open Graph (which is Facebook's system that they standardised and is now widely adopted).

Issue History

Date Modified Username Field Change
2023-01-07 22:25 PDStig New Issue
2023-01-07 22:25 PDStig Status Not Assigned => Assigned
2023-01-07 22:25 PDStig Assigned To => Chris Graham
2023-01-13 14:08 Adam Edington Note Added: 0007867
2023-01-14 23:12 Chris Graham Note Added: 0007869
2023-01-15 11:56 Adam Edington Note Added: 0007873
2023-01-17 00:25 Adam Edington Note Added: 0007875
2023-01-17 02:34 Chris Graham Note Added: 0007876
2023-01-17 03:02 Chris Graham Note Added: 0007877
2023-01-17 03:03 Chris Graham Note Edited: 0007877
2023-01-17 22:02 Guest Note Added: 0007878
2023-01-18 01:44 Adam Edington Note Edited: 0007878 View Revisions
2023-01-18 02:48 Adam Edington Note Added: 0007879
2023-01-18 03:17 Chris Graham Note Added: 0007880
2023-01-18 03:28 Adam Edington Note Added: 0007881
2023-01-18 03:30 Adam Edington Note Edited: 0007881
2023-01-18 03:31 Adam Edington Note Edited: 0007881
2023-01-18 16:37 Chris Graham Note Added: 0007882
2023-01-18 19:29 Adam Edington Note Added: 0007883
2023-02-10 14:30 Chris Graham Note Added: 0007898
2023-02-10 14:30 Chris Graham Status Assigned => Resolved
2023-02-10 14:30 Chris Graham Resolution open => fixed
2023-02-26 18:29 Chris Graham Category General => General / Uncategorised