Skip to content

feat(website): refresh admin film with updated notifications screenshot - #894

Merged
cameri merged 3 commits into
mainfrom
feat/website-admin-film-notification-screenshot
Oct 19, 2026
Merged

cameri merged 3 commits into
mainfrom
feat/website-admin-film-notification-screenshot

Conversation

@Ferryx349

@Ferryx349 Ferryx349 commented Oct 18, 2026 •

Copy link
Copy Markdown
Collaborator

Description

Refresh the home page operator console film so the Notifications / alerts beat uses a new screenshot from the real /admin UI .

Related Issue

Closes :- #893

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Non-functional change (docs, style, minor refactor)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my code changes.
  • I added a changeset, or this is docs-only and I added an empty changeset.
  • All new and existing tests passed.

Re-render and re-encode operator console site media for the alerts beat.
@changeset-bot

changeset-bot Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: ff28c41

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
nostream-website Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@Ferryx349
Ferryx349 deployed to website-preview October 18, 2026 16:19 — with GitHub Actions Active
@github-actions

github-actions Bot commented Oct 18, 2026 •

Copy link
Copy Markdown
Contributor

🧹 Site preview for this PR was deleted (PR closed).

@greptile-apps

greptile-apps Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

RetriggerView in GreptileConfidence Score: 5/5

[Low impact] No new blocking issue was found in this refresh.

Summary

Refreshes the home page’s operator-console film with a new notifications screenshot.

  • The home-page alerts scene now shows an updated admin screenshot.

No new actionable issues were found. The images could not be displayed in this review, and video playback was not verified.

Reviews (2) · Last reviewed commit: "Merge branch 'main' into feat/website-ad..." · Reviewed by Greptile

@greptile-apps

greptile-apps Bot commented Oct 18, 2026

Copy link
Copy Markdown

Comments Outside Diff

These findings could not be posted inline.

  • P2 Console video more than doubles website/site/public/media/admin.mp4:1 ▶

    admin.mp4 grows from 9.5 MB to 22.2 MB for this screenshot refresh. The home page loads it with preload="auto", and the console film has no smaller source for mobile or data-saving users. This adds 12.7 MB per full download and makes playback harder on slow connections. Consider re-encoding the film closer to its previous size.

    Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

@Ferryx349 Ferryx349 changed the title fix(website) : add admin film notification screenshot feat(website): refresh admin film with updated notifications screenshot Oct 18, 2026
@Ferryx349
Ferryx349 deployed to website-preview October 18, 2026 18:40 — with GitHub Actions Active
@cameri
cameri deployed to website-preview October 18, 2026 21:26 — with GitHub Actions Active
@cameri
cameri merged commit 02b70cf into main Oct 19, 2026
14 checks passed
@cameri
cameri deleted the feat/website-admin-film-notification-screenshot branch October 19, 2026 00:21
@cameri
cameri deployed to website-preview October 19, 2026 00:21 — with GitHub Actions Active

This branch was successfully deployed

1 active deployment
website-preview — ff28c41c Deployed Oct 19, 2026 by cameri via preview #89
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