Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.The current Drupal 7 & 8 modules are using the 1.0-beta2 of the Yoast SEO JavaScript. We need to update both with the latest version of the Yoast SEO JavaScript(v.1.2.2)
| Comment | File | Size | Author |
|---|---|---|---|
| #16 | yoast_seo-update-lib-2747541-3.patch | 728 bytes | ozin |
| #13 | interdiff.txt | 4.91 KB | ozin |
| #12 | yoast_seo-update-lib-2747541-2.patch | 240.84 KB | ozin |
Comments
Comment #2
ozinUpdated js with the latest version. Patch attached. Please review.
Comment #3
robertragas CreditAttribution: robertragas commentedHi Ozin, I tested the patch and works great! The new snipper editor is looking good.
It will be included in the next release which I try to do later today.
Comment #5
robertragas CreditAttribution: robertragas commentedComment #6
ozinThanks!
Comment #7
robertragas CreditAttribution: robertragas commentedHmm , reopening it,
The snippet editor works, but I noticed now that it doesn't seem to pick up any of the word within the body field.
Does this works at you?
Comment #8
ozinI will check it. Could you provide more details how it should use body field.
Comment #10
robertragas CreditAttribution: robertragas commentedHello Ozin,
Standard with Yoast it uses the body field for analysis ( there is a patch coming up with multiple field support that your colleague made :) )
Also I checked out some more, and I see this is not the problem after all. The Snippet does take the correct text, the only thing that its missing is that there is no live reload.
In the last version when people fill in some text in the body the SEO score automatic gets updated, but in this one you need to save the node first, then edit it and you will see the correct score.
Do you need more information?
Comment #11
robertragas CreditAttribution: robertragas commentedMaybe it has to do with the added image where u deleted some code that rerenders the snippet from input.
Comment #12
ozinHi @RobertRagas,
I've found some bugs with the title field and rating calculation as well. Please check my patch which should fix all those bugs.
Comment #13
ozinComment #14
robertragas CreditAttribution: robertragas commentedThanks for the new patch ozin, I checked it again and this time everything seems to be working so accepting it and merging it in the branch.
There is 1 minor thing that I noticed. A score of 7 shows in the overview as "ok" with an orange dot, but in the node detail snippet it shows as "good" and green so wonder if we should change the yoast_seo_score_rating to make a 7 show as good.
Comment #16
ozinYes, the problem with the "seo_status" scheme type. There is the patch which fix field type.
Comment #18
robertragas CreditAttribution: robertragas commented