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

lib/gis: markdown parser lowercase and extension fixes #4740

Merged
merged 1 commit into from
Nov 23, 2024

Conversation

neteler
Copy link
Member

@neteler neteler commented Nov 22, 2024

This PR updates the Markdown support in the parser for the manual in terms of

  • converting keywords to lowercase in manual pages so that e.g. dist.x86_64-pc-linux-gnu/docs/mkdocs/site/keywords.html#modis works
  • replacing of .html with .md in certain cases

This PR updates the Markdown support in the parser for the manual in terms of

- converting keywords to lowercase (e.g. `dist.x86_64-pc-linux-gnu/docs/mkdocs/site/keywords.html#modis`)
- replacing of `.html` with `.md` in certain cases
@neteler neteler added manual Documentation related issues docs markdown labels Nov 22, 2024
@neteler neteler added this to the 8.5.0 milestone Nov 22, 2024
@github-actions github-actions bot added C Related code is in C libraries labels Nov 22, 2024
@neteler
Copy link
Member Author

neteler commented Nov 22, 2024

A still open issue is that in the page below the grass-XXX DB pages have a .md extension (should be .html):

dist.x86_64-pc-linux-gnu/docs/mkdocs/site/databaseintro.html
-> Creating a database
---> linked pages in the section have the wrong file extension

All of them start with grass-, maybe that's triggering the issue?

Any ideas, @landam ?

@landam
Copy link
Member

landam commented Nov 22, 2024

A still open issue is that in the page below the grass-XXX DB pages have a .md extension (should be .html):

dist.x86_64-pc-linux-gnu/docs/mkdocs/site/databaseintro.html -> Creating a database ---> linked pages in the section have the wrong file extension

All of them start with grass-, maybe that's triggering the issue?

Any ideas, @landam ?

@neteler Let's continue here: #4742

@echoix
Copy link
Member

echoix commented Nov 23, 2024

I'm letting you two handle this #4740 and #4742 as I'm not sure in what order you'd want them in

@neteler neteler merged commit e7ae0b8 into OSGeo:main Nov 23, 2024
26 of 28 checks passed
@neteler neteler deleted the man_fix_lib_gis_parser_rest_md branch November 23, 2024 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C Related code is in C docs libraries manual Documentation related issues markdown
Projects
Development

Successfully merging this pull request may close these issues.

3 participants