Skip to content

Remove changelog list from the homepage#864

Merged
mfisher87 merged 3 commits intogeojupyter:mainfrom
giswqs:changelog
Aug 20, 2025
Merged

Remove changelog list from the homepage#864
mfisher87 merged 3 commits intogeojupyter:mainfrom
giswqs:changelog

Conversation

@giswqs
Copy link
Collaborator

@giswqs giswqs commented Aug 13, 2025

Description

Resolves #863

Checklist

  • PR has a descriptive title and content.
  • PR description contains references to any issues the PR resolves, e.g. Resolves #XXX.
  • PR has one of the labels: documentation, bug, enhancement, feature, maintenance
  • Checks are passing.
    Failing lint checks can be resolved with:
    • pre-commit run --all-files
    • jlpm run lint

📚 Documentation preview: https://jupytergis--864.org.readthedocs.build/en/864/
💡 JupyterLite preview: https://jupytergis--864.org.readthedocs.build/en/864/lite

@github-actions
Copy link
Contributor

Binder 👈 Launch a Binder on branch giswqs/jupytergis/changelog

@giswqs
Copy link
Collaborator Author

giswqs commented Aug 13, 2025

Contributors can't choose PR label on their own.

@github-actions
Copy link
Contributor

github-actions bot commented Aug 13, 2025

Integration tests report: appsharing.space

Copy link
Member

@arjxn-py arjxn-py left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @giswqs
ReadTheDocs is unhappy for some reason. Pinging @mfisher87 since he can help :)

@mfisher87
Copy link
Member

Contributors can't choose PR label on their own.

I think this is normal GitHub behavior, right? I granted you some more rights so you should be able to label now!

@mfisher87 mfisher87 added the documentation Improvements or additions to documentation label Aug 20, 2025
@mfisher87
Copy link
Member

mfisher87 commented Aug 20, 2025

The problem is that the header navbar is generated from TOC, so removing this toctree directive removes all references to the changelog.md. I think the solution is to make this toctree directive hidden instead of removing it. Then we keep the navbar link to TOC but remove it from the page contents. 🤞

Copy link
Member

@mfisher87 mfisher87 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, Qiusheng!

@mfisher87 mfisher87 merged commit dcac2cc into geojupyter:main Aug 20, 2025
14 checks passed
mfisher87 added a commit to mapninja/jupytergis-docwork that referenced this pull request Aug 28, 2025
* Remove changelog list from the homepage

* Switch to hidden changelog to preserve top navbar link to changelog

---------

Co-authored-by: Matt Fisher <mfisher87@gmail.com>
HaudinFlorence pushed a commit to HaudinFlorence/jupytergis that referenced this pull request Jan 28, 2026
* Remove changelog list from the homepage

* Switch to hidden changelog to preserve top navbar link to changelog

---------

Co-authored-by: Matt Fisher <mfisher87@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove the changelog from homepage

3 participants