3.4.0 Upgrade Support

<baris>

Published 3.4.1 patch version with the pqsl fix.

sweetp

@baris thanks, I just updated to 3.4.1 from 3.3.9 without issue on my production server

DownPW

I just upgraded 3.4.1 from 3.3.9 fine.

I have just this issue : the number of public discussions is not displayed correctly.
For information, my forum is in French language. I didn't notice this problem on my test server which is in English language.

<baris>

@DownPW looks like someone made a typo in the French translation on transifex, it needs to be fixed there. "chat.public-rooms": "Salon Publique (ù1)", should be "chat.public-rooms": "Salon Publique (%1)",

DownPW

I see that two @baris

It seems that I don't have the right icon for the reaction plugin

My:

New on my english development server :

I have uninstall and reinstall the plugin with 2 rebuild :

<baris>

If you are seeing + for reactions you probably have the old plugin installed, maybe check your package.json for nodebb-plugin-reactions and remove it. The latest one is @nodebb/nodebb-plugin-reactions

DownPW

I have the version on the screenshot above

phenomlab

I see an issue on mobile within the chat view. If modals are enabled, chat messages do not open. If modals are disabled, chats open but you can't close the message or use the back button in the message view because it's partially obscured in the sense that the element position is set too high.

Edit - odd, but seems ok now.

DownPW

@baris

package.json at /home/xxxx/nodebb/node_modules/@nodebb/nodebb-plugin-reaction indicates 2.1.9 version

what is the last version ?

<baris>

2.1.10 is latest but 2.1.19 is fine for 3.4.0 as well.

What about package.json in your nodebb folder? It should only have @nodebb/nodebb-plugin-reactions

DownPW

@baris

<baris>

If there is no "nodebb-plugin-reactions" in that file then everything should work and you should see smiley face instead of + after building and restarting

DownPW

So should I delete the line above?

<baris>

@DownPW no that line is fine, if there is a line "nodebb-plugin-reactions": "x.x.x" you should delete that one.

DownPW

nope, I haven't got other line like this but doesn't work grrr

DownPW

I'm going to try uninstalling again

<baris>

You can also check if something has modified topic.tpl like nodebb-plugin-customize, that might prevent the correct reactions template from showing up.

DownPW

yep I use nodebb-plugin-customize for toping rating plugin in topic.tpl file

DownPW

yep @baris I have delete my customisation on topic.tpl file for toping rating plugin and reaction plugin is good now. Thanks.
--> So I'm thinking of deleting this customization if I have to delete it and redo it every time.

-- I also note that the topic rating plugin announces NaN now on topic_list An idea ?

-- So I'm also going to wait for the typo on the french translation. I imagine that we will have to update nodeBB again ?

<baris>

@DownPW @nodebb/[email protected] should fix the NaN issue