Skip to content

Re-structuring of TOC#444

Merged
ronnyhdez merged 23 commits into
NASA-Openscapes:mainfrom
ronnyhdez:I427
Jun 4, 2026
Merged

Re-structuring of TOC#444
ronnyhdez merged 23 commits into
NASA-Openscapes:mainfrom
ronnyhdez:I427

Conversation

@ronnyhdez

Copy link
Copy Markdown
Member

Re-structuring of TOC according to co-working session: #427 (comment)

@ronnyhdez

Copy link
Copy Markdown
Member Author

/deploy-preview

@github-actions

github-actions Bot commented May 7, 2026

Copy link
Copy Markdown

@itcarroll itcarroll added this to the Restructuring Navigation milestone May 7, 2026
@ronnyhdez

Copy link
Copy Markdown
Member Author

/deploy-preview

ronnyhdez added 2 commits May 16, 2026 12:25
is not the fontawesome or bootstrap icon we were initially planning, but is an easy solution compared to change the css style or implementing html syntax in the file
@ronnyhdez ronnyhdez linked an issue May 16, 2026 that may be closed by this pull request
@ronnyhdez ronnyhdez requested a review from ateucher May 19, 2026 16:22
@ateucher

Copy link
Copy Markdown
Member

/deploy-preview

@ateucher

Copy link
Copy Markdown
Member

@ronnyhdez I triggered the deploy preview with the /deploy-preview comment. The reason it wasn't running on your subsequent commits is because your PR is from an external fork, not a branch. When that's the case, the /deploy-preview comment is necessary whenever you need to create/update the preview. See #458 to hopefully make this clearer

@ateucher ateucher left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@ronnyhdez I think this looks great! I think you captured the structure we talked about pretty much perfectly. I have a few comments/suggestions/questions, but all pretty minor.

I don't know if we want to merge this yet, or see if we can figure out a way to deploy to a /latest endpoint first, so we don't keep changing the stable cookbook while we continue to iterate? See #439 by @itcarroll

Comment thread _quarto.yml
Comment thread _quarto.yml
Comment thread _quarto.yml
Comment thread _quarto.yml
Comment thread _quarto.yml Outdated
Comment thread _quarto.yml Outdated
Comment thread _quarto.yml
Comment thread _quarto.yml Outdated
Comment thread _quarto.yml
@ronnyhdez ronnyhdez linked an issue May 24, 2026 that may be closed by this pull request
The files will be re-classified according to its content in the next milestone
@ronnyhdez

Copy link
Copy Markdown
Member Author

/deploy-preview

@ronnyhdez

Copy link
Copy Markdown
Member Author

Hi @itcarroll, @alexishunzinger, @danielfromearth!

Here is the PR with the changes we have been discussing in the latest co-working sessions. @ateucher has already reviewed the content and I have addressed all the comments. 😄

The main intention of this PR is to capture the proposed new main sections and their contents. Most of the changes are in _quarto.yml, so the impact is mainly on the table of contents structure rather than modifying or deleting existing files.

Changes to the actual files (i.e. removing a WIP content and instead creating a new issue or a PR) would happen in the next stage.

When you have a chance, could you please take a look and let me know if you agree with this structure, or if there is anything missing or that should be changed?

Thanks!

@itcarroll itcarroll left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Minor change requests, but otherwise looks correct to me.

There are many links that are now incorrectly getting the external link icon, which I think is being worked on in a separate PR, but perhaps that is a preview artifact.

Comment thread _quarto.yml Outdated
Comment thread _quarto.yml Outdated
Comment thread _quarto.yml
@betolink

betolink commented Jun 3, 2026

Copy link
Copy Markdown
Member

I think the ASF NISAR user guide should be included here? cc @jhkennedy

@jhkennedy

Copy link
Copy Markdown
Contributor

I think the ASF NISAR user guide should be included here? cc @jhkennedy

Oh yes, that would be good to include. Could add it here (I'll add a suggestion) or as a sepearte PR.

Comment thread _quarto.yml
Comment thread _quarto.yml
Comment thread _quarto.yml
Comment thread _quarto.yml Outdated
Comment thread _quarto.yml Outdated
ronnyhdez and others added 6 commits June 4, 2026 10:01
Co-authored-by: Daniel Kaufman <114174502+danielfromearth@users.noreply.github.com>
Co-authored-by: Joseph H Kennedy <me@jhkennedy.org>
Co-authored-by: Daniel Kaufman <114174502+danielfromearth@users.noreply.github.com>
Co-authored-by: Daniel Kaufman <114174502+danielfromearth@users.noreply.github.com>
Co-authored-by: Ian Carroll <carroll.ian@gmail.com>
Co-authored-by: Ian Carroll <carroll.ian@gmail.com>
@ronnyhdez ronnyhdez merged commit 98def13 into NASA-Openscapes:main Jun 4, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

7 participants