nodebb-plugin-category-sections - ACP still broken

pichalite

@ThingBreaker may be there is some error if using Redis as DB. I have no problems running it using Mongo.

Will try to spin a new instance using Redis and see if it works.

? Offline

@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.

pichalite

@ThingBreaker I am getting the same error if I use Redis. not sure what the fix is...

pichalite

@ThingBreaker I think I found a fix for this...

remove existing plugin settings from Redis (if any saved)

connect to redis using redis-cli and run DEL plugins:category-sections:sections to remove any saved settings

add 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

Jason Neal

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.

pichalite

@Jason-Neal already sent the PR to the repository. no need to do it again.

pichalite

@Jason-Neal @ThingBreaker new version of plugin published with compatibility for NodeBB v1.x.x and fix for settings save when using redis.

Jason Neal

@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!

Danny McWilliams

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?

phit

@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

pichalite

@phit @Danny-McWilliams plugin used to support persona but recent changes to persona/templates.js made it incompatible.