nodebb-plugin-category-sections - ACP still broken
-
@pichalite, I'd be curious to hear your results. I haven't worked with mongo previously though it seems to be the new standard for NodeBB installations.
-
@ThingBreaker I am getting the same error if I use Redis. not sure what the fix is...
-
@ThingBreaker I think I found a fix for this...
remove existing plugin settings from Redis (if any saved)
connect to redis using
redis-cliand runDEL plugins:category-sections:sectionsto remove any saved settingsadd the following code before this line https://github.com/NodeBB/nodebb-plugin-category-sections/blob/master/static/lib/admin/sections.js#L66
sections = JSON.stringify(sections);restart NodeBB
-
I have to say, I'm loving the active community here for NodeBB. Reminds me of when SMF was still pretty new.
Would it help if I forked the repository, made the fix and asked for a pull into the master?
Not sure about the order of operations expected here in the community.
-
@Jason-Neal already sent the PR to the repository. no need to do it again.
-
@Jason-Neal @ThingBreaker new version of plugin published with compatibility for NodeBB v1.x.x and fix for settings save when using redis.
-
@pichalite You're awesome man.
Very excited about nodebb and what it can do.
I look forward to becoming an active part of the community here.
Kudos to @psychobunny for all of the AWESOME plugins he's developed here too!
-
Don't think starting a new topic is necessary for my question - is this only compatible with Lavender and if not is there a way of doing this with Persona?
-
@Danny-McWilliams said:
is this only compatible with Lavender and if not is there a way of doing this with Persona?
seems to be compatible with persona, but I haven't tried it
https://github.com/NodeBB/nodebb-plugin-category-sections/blob/master/library.js#L136 -
@phit @Danny-McWilliams plugin used to support persona but recent changes to persona/templates.js made it incompatible.