joplin / plugin-bibtex

Use locally stored BibTeX files to integrate citation into Joplin
30 stars 7 forks source link

Parser implementation & Testing #2

Closed xUser5000 closed 3 years ago

xUser5000 commented 3 years ago

What has been done

laurent22 commented 3 years ago

That looks good. Only thing I'm wondering is how complex is the data you will need for the plugin? I guess you'll ignore a good part of what's in the bibtex data - for example you won't display the isbn or issn, etc. In that case, what I'd suggest is create a TypeScript interface that contains the property you'll need, and the parse method would return this structure.

I expect it's quite easy to write this interface, and doing so will have many benefits:

If you need more info about this let me know. You don't need to add the interface in this pull request, it can be in a separate one.