- minor: `ForgeTracker/forgetracker/templates/tracker/milestone.html` has
`import 'allura:templates/jinja_master/lib.html` twice
- when on a `page=0` url, then I don't see any `rel="next"` tag. Remember
page=0 means first page in allura! :grin:
- when on a `page=1` url then the `<link rel="prev"
href="/p/test/tickets/milestone/1.0/"/>` doesn't have the fully-qualified URL
with domain
- also when on a ticket search then this `rel="prev"` tag loses the
`?q=...` part of its URL
- the `def querystring` helper is a little complicated, can we have a docstring
to explain what it does? Like "make a new URL based on the current request and
new params" if that is a correct explanation :smile: Maybe even a unit test
too? (you can make a fake request with `Request.blank`
- I think there's a mismatch between passing `count` into
`lib.pagination_meta_tags(` and that macro taking the parameter as `last_page`.
The count of items is not equal to the last page number. Need to divide by
the `limit` size I think to convert `count` into `last_page`.
- probably should include these tags on the main ticket list and the blog main
view too.
---
** [tickets:#8444] Add Canonical Link For Tool Sections **
**Status:** open
**Milestone:** unreleased
**Created:** Tue Jun 28, 2022 05:50 PM UTC by Guillermo Cruz
**Last Updated:** Tue Jun 28, 2022 05:50 PM UTC
**Owner:** Guillermo Cruz
Tools like Tickets, Wikis, Discussions, etc. should include a canonical link.
Also on sections with pagination include `page=x` in the root url and
`rel="next"` `rel="next"` tags
---
Sent from forge-allura.apache.org because [email protected] is subscribed
to https://forge-allura.apache.org/p/allura/tickets/
To unsubscribe from further messages, a project admin can change settings at
https://forge-allura.apache.org/p/allura/admin/tickets/options. Or, if this is
a mailing list, you can unsubscribe from the mailing list.
