mProjectsCode / obsidian-js-engine-plugin

https://www.moritzjung.dev/obsidian-js-engine-plugin-docs/
GNU General Public License v3.0
73 stars 5 forks source link

[FR] Remove unnecessary log #19

Closed Mara-Li closed 1 month ago

Mara-Li commented 1 month ago

Additional FR here : https://github.com/mProjectsCode/obsidian-meta-bind-plugin/issues/384

The plugin seems to print the content of each blocks in the console, that leads to an enormous content of the console. Duplicated sometomes by meta-bind itself. Sometimes, the amount of the logs can be more than 1000 lignes.

Plugin guildelines state we should avoid unncessary logging to the console, so the plugin should be updated in accordance.

mProjectsCode commented 1 month ago

I am not 100% sure what you are referring to. The plugin logs the markdown string it renders here (this should probably be removed or turned into a debug log).

Mara-Li commented 1 month ago

I made a draft PR to show you the faulty line that create the log spamming issue

mProjectsCode commented 1 month ago

this should all be done. Thanks for the issue and PR