Skip to content

GA cheatsheet#29

Closed
padmashris wants to merge 6 commits into
mainfrom
padma_updates
Closed

GA cheatsheet#29
padmashris wants to merge 6 commits into
mainfrom
padma_updates

Conversation

@padmashris
Copy link
Copy Markdown
Contributor

No description provided.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 20, 2026

OTTR Check Results

Summary

  • Spelling check: ❌ FAILED (1 errors found, threshold: 0)
  • URL check: ❌ FAILED (5 errors found, threshold: 2)

⚠️ Some Checks Failed Click here to download detailed error reports

Last Updated: 2026-03-30-16:04:35

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 20, 2026

👀 Quick preview of website here *
🔬 Comprehensive download of the website here

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

Updated at 2026-03-30 with changes from the latest commit 4dc6ddb

Padmashri Saravanan added 3 commits February 19, 2026 22:44
- added cheatsheet style Rmd, stored updates to steps in `google_analytics_steps.Rmd`.
- updated GA screenshots in `google_analytics_steps.Rmd`
- adjusted line height and image width
@padmashris
Copy link
Copy Markdown
Contributor Author

padmashris commented Feb 20, 2026

@carriewright11 i added updates to the steps for GA integration here but wanted to check if you wanted me to create a PR on the ottr documentation repo before i put it on there?

the cheatsheet version of it is here

Comment thread css/ITN_style.css
Copy link
Copy Markdown
Contributor

@carriewright11 carriewright11 Mar 31, 2026

Choose a reason for hiding this comment

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

I noticed there are updates to the ITN style - which is for the other cheatsheets (Red and blue) - this is changing the indenting for those which I'd like to keep as it was. There is a OTTR_style file too which is intended for these cheatsheets

Comment thread docs/chunks/secrets.md
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 thought I had added a line to the first chunk to make all the links open in a new tab... so i don't think we need this?

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 is from local rendered files though, so maybe that is why we see this here because it is from the docs dir

Comment thread docs/css/ITN_style.css
Copy link
Copy Markdown
Contributor

@carriewright11 carriewright11 Mar 31, 2026

Choose a reason for hiding this comment

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

I often see the docs folder files getting added ... I wonder if we should add documentation about how this isn't needed? all the rendering should happen through the GitHub action... but maybe things aren't behaving the way I expected.

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.

hmm these would overwrite the generic versions - I think we just wanted to add the Google analytics files, looks like maybe local render files got pushed

Comment thread gallery.yml
href: https://ottrproject.org/cheatsheets/ai_everyday_uses.html
code: https://github.com/ottrproject/cheatsheets/blob/main/ai_everyday_uses.Rmd
thumbnail: pngs/ai_everyday_uses.png
- title: Google Analytics Integration
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.

could we please move this one to the OTTR category - which is below line 40 here

number_sections: false
includes:
in_header: itn_favicon.html
css: css/ITN_style.css
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.

we want to use the ottr style for this one

@padmashris
Copy link
Copy Markdown
Contributor Author

moved to #31, closing this PR now

@padmashris padmashris closed this Apr 22, 2026
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