link olid macro #9348
link olid macro #9348
Conversation
mekarpeles
added
the
On testing.openlibrary.org
openlibrary/macros/OLID.html
Outdated
| $def with (q="#") | ||
|
|
||
| $ edition = query_param('edition', q) | ||
| $ results = find_matches(olid, q) |
There was a problem hiding this comment.
find_matches isn't a public function that's available here. It either needs to be passed in as a parameter, made a @public function, or be added to plugins/openlibrary/code.py as a template global: https://github.com/internetarchive/openlibrary/blob/master/openlibrary/plugins/openlibrary/code.py#L1146-L1171
I think though that find_matches is probably just the wrong function to use. We likely want to use some existing global function to do a web.ctx.site.things get for the specific olid (open library id)
mekarpeles
added
the
Needs: Submitter Input
|
@psyren99 please see code review comments, see if you can find a |
github-actions
bot
removed
the
Needs: Submitter Input
for more information, see https://pre-commit.ci
…y into 2283/linking_to_olid
for more information, see https://pre-commit.ci
cdrini
removed
the
On testing.openlibrary.org
|
Is this still open? |
|
I believe this may need a rebase, then we can throw on testing and if it works, merge :) |
Closes #2283
Create a macro for linking to an OLID
Technical
/macros/OLID.html added
/plugins/upstream/models.py
Testing
/plugins/upstream/tests/test_models.py
Screenshot
Stakeholders