Steps to reproduce:
- Go to https://github1s.com/InfectedLibraries/Biohazrd
- In the file explorer, try to expand
Biohazrd and then #Declarations
Expected behavior:
The folder expands to show its contents:

Actual behavior:
The folder appears to contain the contents of its parent:

Looking at the request log, it appears that the folder name is getting dropped. It sends a request to https://api.github.com/repos/InfectedLibraries/Biohazrd/git/trees/HEAD:Biohazrd/ instead of https://api.github.com/repos/InfectedLibraries/Biohazrd/git/trees/HEAD:Biohazrd/%23Declarations as expected.
I suspect the fix would be to URL encode the path here:
|
return fetch( |
|
`https://api.github.com/repos/${owner}/${repo}/git/trees/${ref}${path.replace( |
|
/^\//, |
|
':' |
|
)}` |
|
).catch(handleRequestError); |
Unfortunately the setup script isn't working on either Windows or WSL for me otherwise I'd try the fix myself.
Love the concept of this project, looking forward to trying it in my workflow!
Steps to reproduce:
Biohazrdand then#DeclarationsExpected behavior:
The folder expands to show its contents:
Actual behavior:
The folder appears to contain the contents of its parent:
Looking at the request log, it appears that the folder name is getting dropped. It sends a request to
https://api.github.com/repos/InfectedLibraries/Biohazrd/git/trees/HEAD:Biohazrd/instead ofhttps://api.github.com/repos/InfectedLibraries/Biohazrd/git/trees/HEAD:Biohazrd/%23Declarationsas expected.I suspect the fix would be to URL encode the path here:
github1s/extensions/github1s/src/api.ts
Lines 69 to 74 in c4886ef
Unfortunately the setup script isn't working on either Windows or WSL for me otherwise I'd try the fix myself.
Love the concept of this project, looking forward to trying it in my workflow!