Tirraa / dashboard_rtm

dashboard_rtm
MIT License
5 stars 1 forks source link

[Content][Computed Fields Builders] Failing with `index.mdx` #68

Closed gustaveWPM closed 9 months ago

gustaveWPM commented 9 months ago

Aouch! This is an edge case!

Gonna track down this.

gustaveWPM commented 9 months ago

After some investigations: this will require an entire dedicated error handling, with checkers before any call to getFlattenedPathWithoutRootFolder, based on the sourceFilePath field of each document.

It must also be file extension agnostic. So any trailing .mdx, .md, .etc will have to be removed from the sourceFilePath during the checks.

gustaveWPM commented 9 months ago

Well at first, I was very depressed. But in the end it is pretty simple to handle, and I think we can expect a merge by the end of the week.

I preferred to create a separate branch because this problem was really stressing me out.

gustaveWPM commented 9 months ago

oops

gustaveWPM commented 9 months ago

Created a new branch: fix-index-notation-lp

gustaveWPM commented 9 months ago

I think this is finally fixed.