(to unstable branch) Adjust opengraph tag to show scrapped image

[Shulammite-Aso]

Fixes #8273
Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!

• PR is descriptively titled 📑 and links the original issue above 🔗
• tests pass -- look for a green checkbox ✔️ a few minutes after opening your PR -- or run tests locally with rake test
• code is in uniquely-named feature branch and has no merge conflicts 📁
• screenshots/GIFs are attached 📎 in case of UI updation
• ask @publiclab/reviewers for help, in a comment below

We're happy to help you get this ready -- don't be afraid to ask for help, and don't be discouraged if your tests fail at first!

If tests do fail, click on the red X to learn why by reading the logs.

Please be sure you've reviewed our contribution guidelines at https://publiclab.org/contributing-to-public-lab-software

Thanks!

[gitpod-io]

[codecov]

Codecov Report

Merging #8291 into unstable will decrease coverage by 0.36%.
The diff coverage is 62.60%.

@@             Coverage Diff              @@
##           unstable    #8291      +/-   ##
============================================
- Coverage     81.68%   81.32%   -0.37%     
============================================
  Files           100      101       +1     
  Lines          5842     5873      +31     
============================================
+ Hits           4772     4776       +4     
- Misses         1070     1097      +27     

Impacted Files	Coverage Δ
app/controllers/spam2_controller.rb	61.53% <7.14%> (-14.06%)	⬇️
app/controllers/batch_controller.rb	67.96% <67.96%> (ø)
app/controllers/tag_controller.rb	81.40% <100.00%> (+0.05%)	⬆️
app/controllers/wiki_controller.rb	85.22% <100.00%> (+0.22%)	⬆️
app/models/tag.rb	97.47% <100.00%> (ø)

[Shulammite-Aso]

Hi @sagarpreet-chadha @jywarren @cesswairimu can you merge this? Using it to test #8284

[jywarren]

Yep! Done! Thanks!!!!

[Shulammite-Aso]

This didn't work.🙊

I guess i'ld have to request from @cesswairimu or anybody who knows about how to access an image(preferably the first image) from the post's body from this file.

Also perhaps default to the public lab logo when the post has no image at all: main_image and post body image. Will this be okay?

[sagarpreet-chadha]

Hey @Shulammite-Aso ,
Can you look at this amazing data model representation to learn the relationships between tables in the DB: https://github.com/publiclab/plots2/blob/main/doc/DATA_MODEL.md
So from the image in the above link we can see that each node has many images which we can see in node.rb, line:

plots2/app/models/node.rb

Line 91 in cc59cda

has_many :images, foreign_key: :nid

So once you have reference of that node ruby object, using images getter function we can get all the images, right.

Thanks! 😄

Also in the documentation of twitter it is mentioned that the images size should be of certain size and ratio: https://developer.twitter.com/en/docs/twitter-for-websites/cards/overview/summary-card-with-large-image

We can test the images displayed using these tools provided by FB and twitter:
https://developer.twitter.com/en/docs/tweets/optimize-with-cards/guides/getting-started
https://developers.facebook.com/docs/sharing/webmasters/

[sagarpreet-chadha]

Also you can use gitpod url to test instead of pushing to unstable:
https://3000-c5782e9a-bc16-446a-a16e-e2eaaaf4ab1c.ws-us02.gitpod.io/wiki/licenses