[nodebb-plugin-glossary] Glossary Plugin for NodeBB

[phenomlab]

@baris I found the issue (for my case). In library.js the regex statement looks for insensitive as below

keyword.nameRegex = XRegExp(`(?:^|\\s|\\>|;)(${keyword.name})`, 'gi');

I've changed it to

keyword.nameRegex = XRegExp(`(?:^|\\s|\\>|;)(${keyword.name})`, 'g');

Which works fine for my use case (virtually all of the acronyms I have are in upper case). However, lower case words are still affected.

https://github.com/NodeBB/nodebb-plugin-glossary/issues/3

phenomlab created this issue in NodeBB/nodebb-plugin-glossary

closed Issue with over gratuitous regex #3

[baris]

Seems like it is already matching the whole world AI it just matches lowercase as well. I can make the case-sensitivity an option in the ACP page.

[phenomlab]

@baris Yep, that would be great..

[phenomlab]

@baris From what I see, the plugin is actually firing on whole words

In this case, it's matching on bits as well as bit (which it should)

And also matching on dev inside devil

EDIT: Sorry - this was due to the CSV import I have... Ignore that

[phenomlab]

@baris another suggestion would be to empty the table storing the glossary items. When you import 1000 plus from a CSV it's very clunky to have to delete them all one by one if you need to.

https://github.com/NodeBB/nodebb-plugin-glossary/issues/5

phenomlab created this issue in NodeBB/nodebb-plugin-glossary

closed Allow option to empty glossary content #5

[baris]

Published 0.0.7, with bunch of fixes and new options.

[phenomlab]

@baris Thanks. I spotted a bug in 0.6 when performing a large import. The mapping being returned should be (for example SNMP Simple Network Management Protocol comes back with SNMP something completely wrong)

Examples

and

Not sure if my data is incorrect (I'll check), but wanted to report this in case it is actually an issue. Reverting back to 0.5 the issue is not present

and

So, I think there is a bug somewhere

[baris]

Yeah the csv import is pretty picky, it expects the format to be

keyword1, description1 
or 
keyword2, "description2"

If you provide a small csv sample that doesn't work I can take a look.

[baris]

https://github.com/NodeBB/nodebb-plugin-glossary/issues/7

barisusakli created this issue in NodeBB/nodebb-plugin-glossary

closed add csvtojson support #7

[phenomlab]

@baris thanks. So a space is required between the comma and the next field. Odd that this doesn't seem to happen in 0.0.5

[baris]

I updated the plugin to use https://www.npmjs.com/package/csvtojson so it is using the csv spec from https://www.loc.gov/preservation/digital/formats/fdd/fdd000323.shtml

[phenomlab]

@baris This doesn't seem to work for me at all using the latest version sadly. It doesn't matter which format I use, the import always seems to work, but exhibits the same issues as I highlighted earlier. Version 0.5 does not have these issues from what I can tell.

[baris]

Please share the csv you are having an issue with.

[phenomlab]

@baris here you go

csv

It's worth noting that after the import, I then backported to 0.0.5 and the import works as intended.

[baris]

Published, 0.0.10 with some more fixes. I tried your csv and SNMP is working with latest. Obviously I haven't tested all 1000+ keywords

[phenomlab]

@baris thanks very much for this. I'll test it now

Edit - from what I can see, everything now works as expected. Thanks

[baris]

Published 0.0.16

• New /glossary page to show a list of keywords
• Ability to change or remove the icon displayed next to the keyword
• Tooltips will work outside of topic pages ie on user profiles or recent cards plugin

[phenomlab]

@baris not seeing this in npm yet ?

[yLothar]

@baris now that I know the trick...

[baris]

Added i18n, feel free to send pull requests to add new languages. https://github.com/NodeBB/nodebb-plugin-glossary/commit/616ab500cd6ad906ace7eb7510f7f08789d3515d

0 barisusakli committed to NodeBB/nodebb-plugin-glossary

feat: add i18n