-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
Opengraph, canonical URLs, 'next' and 'prev' #1448
Opengraph, canonical URLs, 'next' and 'prev' #1448
Conversation
templates/category/index.html
Outdated
<meta property="og:description" content="{{ category.description }}"> | ||
<meta property="og:image" content="{% static "images/block1.png" %}"> | ||
|
||
{% with category_url=category.get_full_absolute_url %} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You have access to request
here which can convert relative URLs to absolute ones, you don't need category.get_full_absolute_url
.
saleor/product/models.py
Outdated
def get_full_absolute_url(self, ancestors=None): | ||
return build_absolute_uri( | ||
self.get_absolute_url(ancestors=ancestors) | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pretty pleeeeeease with a cherry on top: don't leave hanging parenthesis, brackets or braces.
f07490f
to
14e5d7e
Compare
Codecov Report
@@ Coverage Diff @@
## master #1448 +/- ##
==========================================
+ Coverage 82.44% 83.54% +1.09%
==========================================
Files 131 128 -3
Lines 5668 5784 +116
Branches 647 662 +15
==========================================
+ Hits 4673 4832 +159
+ Misses 831 794 -37
+ Partials 164 158 -6
Continue to review full report at Codecov.
|
d66947a
to
5199b1b
Compare
Close #1403
Close #1345
canonical
forProduct
andCategory
pagesnext
andprev
tags forCategory
andSearch
pagesTODO:
next
andprev
tags forCategory
pageog:url
links absolute