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

PLANET-5388 Fix author bio translation on Post and Author page #1207

Merged
merged 1 commit into from Oct 20, 2020

Conversation

sagarsdeshmukh
Copy link
Member

@sagarsdeshmukh sagarsdeshmukh commented Oct 20, 2020

Rel. to JIRA 5388

  • Fix author bio translation on Post
  • Fix author page meta data & social share button which were showing english author bio on non english author page

eg. https://k8s.p4.greenpeace.org/test-nix/fr/story/43917/quest-ce-que-le-lorem-ipsum-fr/

@sagarsdeshmukh sagarsdeshmukh requested a review from a team October 20, 2020 09:57
@sagarsdeshmukh sagarsdeshmukh self-assigned this Oct 20, 2020
@sagarsdeshmukh sagarsdeshmukh requested review from lithrel and mleray and removed request for a team October 20, 2020 09:57
@@ -12,11 +12,12 @@
{% endset %}

{% set author_bio_char_limit = 180 %}
{% set post_author_description = fn('the_author_meta', 'description', post.author.id) %}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's get_the_author_meta (missing get_) ? Doesn't return anything without it on my env

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah, actually it's working inside twig, but using get_the_author_meta () is more appropriate here 👍 ( In source code the the_author_meta() function internally calls the get_the_author_meta() function. )

Copy link
Member

@lithrel lithrel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants