Skip to content

Set up for Basic Quarto Site#1

Merged
kweav merged 23 commits into
mainfrom
kweav-setUp
Sep 26, 2025
Merged

Set up for Basic Quarto Site#1
kweav merged 23 commits into
mainfrom
kweav-setUp

Conversation

@kweav

@kweav kweav commented Jul 25, 2025

Copy link
Copy Markdown
Contributor
  1. Updating the site navigation per issue discussion in ottrproject.org repo
  2. Edited the info within index.qmd to match up with the info from ottproject.org
  3. Brought over the icons and the setup for the icons from the cheatsheets
  4. Brought over the basic ottr on the water icon and favicon
  5. Added an outline for some docs but haven't ported over material yet. These include
  • getting_started.qmd
  • next_steps.qmd
  1. Directly brought over some docs from ottrproject.org but need some updating to include quarto information, etc.. These include
  • customize-robots.qmd
  • customize-style.qmd
  • more_features*.qmd
  • ottr-fy.qmd
  • faqs.qmd
  • examples.qmd
  1. Brought over all the images from ottrproject.org repo -- will go through later to keep only what we need/update

@github-actions

github-actions Bot commented Jul 25, 2025

Copy link
Copy Markdown
Contributor

OTTR Check Results

Summary

  • Spelling check: ✅ PASSED (0 errors found, threshold: 0)
  • URL check: ✅ PASSED (0 errors found, threshold: 0)
  • Quiz format check: ✅ PASSED (0 errors found, threshold: 0)

🎉 All checks passed!

Last Updated: 2025-09-26-20:04:17

@github-actions

github-actions Bot commented Jul 25, 2025

Copy link
Copy Markdown
Contributor

Re-rendered previews from the latest commit:

* note not all html features will be properly displayed in the "quick preview" but it will give you a rough idea.

Updated at 2025-09-26 with changes from the latest commit 0ad08e2

@kweav kweav requested a review from carriewright11 July 25, 2025 19:27
@kweav

kweav commented Jul 25, 2025

Copy link
Copy Markdown
Contributor Author

Still need to address the spell and URL check and update content throughout (and delete unnecessary images), but how does this look style and layout wise? @carriewright11

@kweav

kweav commented Jul 25, 2025

Copy link
Copy Markdown
Contributor Author

My observations/notes:

  • the icons from _quarto.yml aren't showing up in the navigation -- fixed this with a quarto extension in commit 8163676
  • the spacing of the title is weird -- I think I've fixed this by making the icons work
  • I'm not sure what the furthest left "OTTR" is for in the header -- I think I fixed this by increasing the icon size in commit b911edc
  • the right hand side "on this page" TOC is weird. Would left side be better if I can move it there?

@kweav

kweav commented Jul 25, 2025

Copy link
Copy Markdown
Contributor Author

The "OTTR" on the left takes you to the homepage, so I think we might have to look into fixing/reordering the title (and icon) with respect to the homepage:
image

@kweav

kweav commented Jul 28, 2025

Copy link
Copy Markdown
Contributor Author

Using this quarto extension, I've got icons in the navbar: https://github.com/quarto-ext/fontawesome (documentation), https://quarto.org/docs/extensions/managing.html (installation instructions and quarto itself suggesting it)

@kweav

kweav commented Aug 22, 2025

Copy link
Copy Markdown
Contributor Author

@carriewright11 is this good to merge so that future PRs start changing/adding material? This one got a little big file wise.

Comment thread index.qmd Outdated

@carriewright11 carriewright11 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.

I think this looks great - there are some image paths that might be out-of-date. This is a great start for Allissa. I might tweek the front page in another PR with the benefits image. sound good @kweav ?

Comment thread customize-robots.qmd
```

2) The other URL check runs on a set interval to see if any URLs referenced are no longer valid. This one is set by:
```

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.

Suggested change
```

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.

was trying to suggest adding a space here between the code and like 92... but it is previewing the suggestion in a strange way

Comment thread customize-robots.qmd Outdated

The PR comment included below shows an example of a check that passed (spell check), and a check that failed (the URL check) with the option to download detailed error report(s).

![](https://raw.githubusercontent.com/ottrproject/ottrproject.github.io/kweav-actionsDocsUpdate/resources/screenshots/check_results.png)

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.

not sure if these paths are up-to-date?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

You're correct -- they were wrong. I've updated them in this file. Looking at other files. Right now they're pointing to the old repo. I think we'll want to streamline what we have in images and point to them in this repo once they're on main (as explained in my notes above as future work)

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.

cool I think this could be good to merge if you agree

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yeah -- I agree. Going to convert the notes to issues and then merge

updating image paths to main on other repo
Comment thread more_features_ganalytic.qmd
Co-authored-by: Carrie Wright <23014755+carriewright11@users.noreply.github.com>
Comment thread customize-robots.qmd
### Spell checking

In the `config_automation.yml` file it is set by:
```

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Suggested change
```

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@carriewright11 I tried to make your change in a similar spot and it's doing the same weird preview of the suggestion. So I'm going to make the changes locally and push them

@kweav

kweav commented Sep 26, 2025

Copy link
Copy Markdown
Contributor Author

I've updated the issues tab in this repo to reflect the things we've notated here that need updates going forward. Going to merge

@kweav kweav merged commit 27c893d into main Sep 26, 2025
5 checks passed
@kweav kweav deleted the kweav-setUp branch September 26, 2025 20:56
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.

2 participants