Graph-Link-Types icon indicating copy to clipboard operation
Graph-Link-Types copied to clipboard

Do not stop checking DataView properties on first failure

Open MarkCDavid opened this issue 1 year ago • 5 comments

A small fix for the following situation:

[what_is_this:: not_a_link]
[what_is_that:: [[that_is_a_link]]]

When iterating through the DataView properties, previously, the code would see that the metadata property what_is_this is not a link and would stop immediately. If we allow for it to continue, instead of stopping, it will check what_is_that property, return it as intended and render the link correctly.

The fix here allows for consideration across all DataView properties, as such, we can have DataView properties with a link and ones without a link together.

MarkCDavid avatar Dec 22 '24 23:12 MarkCDavid

Thanks, this helped

dobien avatar Jan 04 '25 19:01 dobien

@MarkCDavid Thanks a lot! I also stumbled upon this after freshly installing Graph Links Types and scratching my head on why it didn't work...

@natefrisch01 Thank you for the great plugin! Would you be able to merge this fix so that new users do not hit this roadblock in the future? Thanks!

cr7pt0gr4ph7 avatar Feb 26 '25 15:02 cr7pt0gr4ph7

I too just hit this problem. If I go to a note where things aren't working... if I delete the headers and a lot of the text, it starts working and adding everything back leaves everything operational. Does that help?

Also... @MarkCDavid , it seems you may have found a solution that was approved by @cr7pt0gr4ph7 ? Can I install aversion from your branch?

DataStrategist avatar Jul 21 '25 13:07 DataStrategist

I have the same issue and it's quite frustrating after I spent an afternoon figuring out why it just doesn't work.

cx-diao avatar Sep 18 '25 19:09 cx-diao

I've tried to include this change on my fork here https://github.com/CHerSun/Graph-Link-Types . ~~Can't seem to get this plugin to work at all, whatever I do - I can't get link descriptions shown (both on local graph and global one)~~ my bad, forgot to reenable it after new installation. Works as expected.

In the fork I've tried to do the upkeeping work - bumped package versions, added automatic pipeline to build release and bump plugin versions, but my knowledge in JS/TS is definitely lacking to actualize the code itself. I'd appreciate the help to revive the plugin and maybe extend functionality.

Plugin from my fork can be added via BRAT using the github link.

CHerSun avatar Nov 12 '25 23:11 CHerSun