Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Links Are Broken When Links Include Uppercase Characters #15

Closed
PSchmiedmayer opened this issue Aug 24, 2021 · 0 comments · Fixed by #17
Closed

Links Are Broken When Links Include Uppercase Characters #15

PSchmiedmayer opened this issue Aug 24, 2021 · 0 comments · Fixed by #17

Comments

@PSchmiedmayer
Copy link

PSchmiedmayer commented Aug 24, 2021

Current Issue

If the name of the type or the target's name contains upper case characters results in links generated by docc2html results in 404 errors.
docc2html generates an html page for a taget named TargetName, type name TypeName, and function name test at the following path: /documentation/targetname/typename/test(_:).html even though the links embedded in other pages look like this /documentation/TargetName/TypeName/test(_:).html.

You can reproduce the issue in the following GitHub Pages based deployment: https://apodini.github.io/Template-Repository/documentation/apodinitemplate/apodinitemplate/index.html
The link to the greet function in the HTML page is https://apodini.github.io/Template-Repository/documentation/ApodiniTemplate/ApodiniTemplate/greet(_:).html while the HTML page can be found at https://apodini.github.io/Template-Repository/documentation/apodinitemplate/apodinitemplate/greet(_:).html

Related Issues

The issue was first noted in issue #13

@PSchmiedmayer PSchmiedmayer changed the title Links Are Broken When Targets Include Uppercase Characters Links Are Broken When Links Include Uppercase Characters Aug 24, 2021
marcprux added a commit to fair-ground/Fair that referenced this issue Nov 3, 2021
@helje5 helje5 closed this as completed in #17 Jan 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging a pull request may close this issue.

1 participant