https://www.dendron.so/ logo
a

andrey-jef

02/09/2022, 9:37 AM
Today I’ve just encountered this error when navigating a Dendron published page. Situation. Open a parent note on ios safari. Then click to open a child note. Next click back to previous page. Then I saw this error page. Is this a known bug?
k

kevins8

02/09/2022, 4:11 PM
do you have a url for this?
a

andrey-jef

02/09/2022, 6:50 PM
yes. here it is the url of the parent node https://kool.casa/notes/6kXK6xVm8TcBoJCwXHkN6/ then u can click on any children node. and go back. The error is on safari or mobile browser on ios only. I tried Brave and Chrome.
j

jyeung

02/10/2022, 1:05 AM
I can repro this on desktop/edge. Console output attached - looks like issue is with the tree view.
Copy code
TypeError: Cannot read properties of undefined (reading 'id')
    at Function.s.getAllParents (_app-fea87210fe336199.js:1:1114930)
k

kevins8

02/10/2022, 5:40 PM
hey @User - is this available as a github repo? if so, i can pull down and inspect
a

andrey-jef

02/10/2022, 6:45 PM
U mean repo of the vault? Sure i will dm u
k

kevins8

02/11/2022, 12:40 AM
status update -> i've repro'd and and root caused the issue. we will prioritize a fix to go out next week
the reason this error happens is because netlify lower cases all urls and dendron's urls are case sensitive. see https://www.jvt.me/posts/2019/11/11/gotcha-netlify-lowercase/
j

jyeung

02/11/2022, 3:45 AM
@User - we took a look at this, we'll try to add a fix to where we treat ID's with case-invariance. If you want to unblock immediately, one (albeit painful) thing you can do is lowercase all your note ID's. i.e.
id: wT4rqJdMxq5cm5LHyQwoD
->
id: wt4rqjdmxq5cm5lhyqwod
. VS Code has a command to help with this -
Transform to Lowercase
in the command palette
a

andrey-jef

02/11/2022, 4:38 AM
Great. Thank you two for investigating. It’s my joy to learn the reason and possible fix like this. I’m also curious if you treat id as case insensitive, do u need to increase the length of id string, in order to keep the same quantity of possible distinct uid?
j

jyeung

02/11/2022, 5:08 AM
we were also discussing that - did some quick math, I think it's still fine with the current length; collision probability should still be trivially low