Did you update the TC? That’s required too. (The give away is lack of localisation that only the latest TC has)
There is specific guidance not to update the plugin outside of the first 5 days of the month. Yes the main plugin is not compatible with current Discourse. This guidance applies to all Pavilion plugins (the supported upgrade period is the first five days)
We’re a core change ‘taker’ so we need time to update the plugin after breaking changes in core. We focus on this at the beginning of the month. We still have to make a living though so we don’t guarantee compatibility at other times. If you upgrade only at the beginning of the month this should be a non-issue.
thank you for information.
The experimental Theme Component is currently working.
That’s all kinds of not right. Not sure what’s wrong with your setup but that’s definitely not the latest TC code running. Odd!
I have it on two production sites and the thumbnail picker is working perfectly on both (albeit it excludes very small images at mo)
Please double check you’ve updated to latest TC.
I was going to start disabling things to figure this out, but switching to a different theme and then switching back somehow fixed it, so odd!
Yeah possibly a caching issue. Good stuff!
Always remember to refresh your webpage after updating a Theme Component. Switching Theme does that for you I suspect.
Next I will be bringing back the action buttons.
Appreciate the testing David!
OK updates to the main plugin to support the new Discourse Thumbnails.
Basically this involves stripping out the code TLP had built to create and manage thumbnails and add new interfaces to work with core thumbnails.
This is a rather large update, but benefits heavily from my work on the TC version:
https://github.com/paviliondev/discourse-topic-previews/pull/151
Because Discourse presently only generates Thumbnails from within Theme Components, you must also add this very simple Theme Component to initiate the thumbnail creation: @David would be great to have a plugin only option here
NB I’m not sure how long the full plugin version will stick around. I’m likely to focus on the Theme Component and the Plugin Sidecar going forward
Please test this out in staging before committing to Production if you can. I have tested it in Production but your mileage may vary. And of course let me know of any issues …
I get this error when installing the generator component
Import Error: about.json does not exist, or is invalid
Let me revert. Thanks for testing.
UPDATE: Apologies! That’s fixed and tested in Production, please update it and refresh your browser.
Moral of the story: don’t release things late in evening when tired! (I had somehow forgotten to push the very file that was critical for it to work!). The readme wasn’t going to get anyone anywhere fast!
OK I’d better summarise the current status as things are getting confusing:
There are now two distinct setups:
- Existing, standard, Plugin based one. I’m thinking of replacing it with 2 though (I will give notice).
-
Plugin
https://github.com/paviliondev/discourse-topic-previews
OR:
- TC based, experimental, expect minor issues but added bonuses of Featured Images on mobile and responsiveness to sidebars … optional plugin sidecar that adds features
-
Theme Component
https://github.com/merefield/discourse-topic-list-previews-theme
-
Add the sidecar plugin if you want more features:
- git clone https://github.com/merefield/discourse-topic-previews.git -b "theme_sidecar"
. This adds ‘actions’ (bookmarking and liking from Topic List) and the Thumbnail Picker in the Topic Meta Editor.
Sorry for any confusion on different setups
The component is live now and it works really well, thanks again!
The only challenge we have now is that "topic list thumbnail resolution level’ affects the featured images as well. So we have either featured images that are a bit too small or thumbnails in the topic list that are too large. Would it be possible to have a “topic list featured images resolution level”?
Edit: And another thing: Apparently, the Topic list preview plugin sorted the featured images by tagging date which makes a lot sense imho. The theme component sorts by last post date which makes the featured images change quite rapidly. Would it be possible to sort by tagging date? Thanks so much!
Good suggestion, that’s implemented: https://github.com/merefield/discourse-topic-list-previews-theme/commit/0b901387eb239e8d1659c0f704fe6754dc4bd8ae
Please update and let me know how you get on.
Works like a charm, thanks so much!
Would it be also possible to sort the featured images by tagging date (see above)?
The plugin has it’s own API changes, unfortunately the TC does not, so not unless I can pull that from a simple existing HTTP request in one go.
Would the publication date work? Since we do a magazine with discourse and your TC, it would be odd, if a user could change the headlines by writing a comment.
Or can you get that feed “/tag/{tag}.json?ascending=false&order=created”?
And the plan is to continue moving forward primarily with the TC, correct?
Yep that could work. And perhaps a setting for latest versus created