Skip to content

updated the largest size files I found in the docs and removed some t…#1367

Merged
thomas-zimmerman merged 5 commits intomasterfrom
resize-large-files
Apr 8, 2026
Merged

updated the largest size files I found in the docs and removed some t…#1367
thomas-zimmerman merged 5 commits intomasterfrom
resize-large-files

Conversation

@Socialhappiness
Copy link
Copy Markdown
Contributor

…ext on the clear cache and cookies article while I replaced the gif with images.

@Socialhappiness Socialhappiness requested review from a team March 19, 2026 17:43
@jacobgkau jacobgkau self-assigned this Apr 2, 2026
Copy link
Copy Markdown
Member

@jacobgkau jacobgkau left a comment

Choose a reason for hiding this comment

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

The old PDFs did not have titles. The new ones have titles that say Untitled, which could be annoying:

Image

It looks like some margins also got added. What tool did you use to compress those?

For PDFs, another solution instead of (or in addition to) shrinking them would be to stick them in the company Dropbox or Google Drive and generate public sharing links. They're already just links, so there's no reason they need to be hosted on the same infrastructure as the website itself.

@jacobgkau
Copy link
Copy Markdown
Member

Replacing the gifs with photos and more descriptive text is a good change.

@Socialhappiness
Copy link
Copy Markdown
Contributor Author

I can't remember what tool I tried, feels like an eternity ago but I think hosting on Drive and linking was a good call and I believe this latest commit achieves that

Copy link
Copy Markdown
Contributor

@thomas-zimmerman thomas-zimmerman left a comment

Choose a reason for hiding this comment

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

Live build of the page looks good. Used a link checker and all of the links come back good:

linkchecker --check-extern http://localhost:3000/articles/guides
Statistics:
Downloaded: 147.50KB.
Content types: 1 image, 94 text, 0 video, 0 audio, 10 application, 0 mail and 0 other.
URL lengths: min=20, max=115, avg=68.

That's it. 105 links in 119 URLs checked. 14 warnings found. 0 errors found.
Stopped checking at 2026-04-03 15:32:05-006 (36 seconds)

All of the warnings are for links out to the rest of the S76 site that have 301 redirects to the "real" site, not on the NPM served version.

@jacobgkau jacobgkau force-pushed the resize-large-files branch from 04dd35f to f16c545 Compare April 7, 2026 20:27
Copy link
Copy Markdown
Member

@jacobgkau jacobgkau left a comment

Choose a reason for hiding this comment

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

The Drive links appear to be working (and do not require logging into Google).

The updated content looks good. I simplified the new Guides page's verbiage and organization a little.

Copy link
Copy Markdown
Contributor

@thomas-zimmerman thomas-zimmerman left a comment

Choose a reason for hiding this comment

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

...and second approval.

@thomas-zimmerman thomas-zimmerman merged commit 314d7a1 into master Apr 8, 2026
3 checks passed
@thomas-zimmerman thomas-zimmerman deleted the resize-large-files branch April 8, 2026 17:51
@jacobgkau
Copy link
Copy Markdown
Member

Before this PR:

Warning: Uploaded artifact size of 1456347784 bytes exceeds the allowed size of 1 GB. Deployment might fail.

After this PR:

Warning: Uploaded artifact size of 1152606491 bytes exceeds the allowed size of 1 GB. Deployment might fail.

Still more reduction to be done, but we're much closer to the warning level now.

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.

3 participants