Skip to content

Conversation

@gisellerosetta
Copy link
Contributor

@gisellerosetta gisellerosetta commented Oct 13, 2025

Proposed Changes

Related Issues

Pre-requisites

This section may not be fully required if the branch is not merging into main.
Please indicate items that aren't necessary and why, with comments around incomplete checks.

  • Version number has been incremented, according to SemVer NA
  • Changelog has been updated, listing changes to this version. Use the keep a changelog format
  • New features are tested NA
  • New features are documented using the numpydoc docstring format NA
  • Other relevant package documentation is updated
  • For new functionality, examples are included in the docs or a feature request has
    been made for it/them. NA

@gisellerosetta gisellerosetta self-assigned this Oct 13, 2025
@gisellerosetta gisellerosetta linked an issue Oct 13, 2025 that may be closed by this pull request
Copy link

@CHCRowley CHCRowley left a comment

Choose a reason for hiding this comment

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

This is looking great! I'm finding it much more logical to navigate. Small thing but you could add additional subheadings to the 'Getting Started' page for the Notes and Cover page examples. Also I remember seeing an example for adding multiple sheets I'm not sure if this needs to be included somewhere? Otherwise happy to approve 😄

@CHCRowley CHCRowley self-requested a review October 14, 2025 08:07
Copy link

@CHCRowley CHCRowley left a comment

Choose a reason for hiding this comment

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

This works really well 😄

@gisellerosetta gisellerosetta merged commit fe5cf72 into dev Oct 14, 2025
11 of 12 checks passed
@gisellerosetta gisellerosetta deleted the 287-restructure-docs branch October 14, 2025 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Review website documentation for clarity

3 participants