Skip to content

Hotfix static render 10y page#15899

Merged
pettinarip merged 3 commits into
masterfrom
hotfix-10y
Jul 21, 2025
Merged

Hotfix static render 10y page#15899
pettinarip merged 3 commits into
masterfrom
hotfix-10y

Conversation

@pettinarip
Copy link
Copy Markdown
Member

Description

  • renders page statically for faster loads
  • get current holder from the transfer events to reduce temporary discrepancies between the 2 data sources (contract & etherscan api)

@netlify
Copy link
Copy Markdown

netlify Bot commented Jul 21, 2025

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit ab6f2f4
🔍 Latest deploy log https://app.netlify.com/projects/ethereumorg/deploys/687e2d56f1a4b000083a6ca4
😎 Deploy Preview https://deploy-preview-15899--ethereumorg.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
7 paths audited
Performance: 52 (no change from production)
Accessibility: 94 (no change from production)
Best Practices: 92 (no change from production)
SEO: 99 (no change from production)
PWA: 59 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions Bot added the tooling 🔧 Changes related to tooling of the project label Jul 21, 2025
Comment thread src/lib/torch/index.ts
{ revalidate: 86400 }
)
export const getCurrentHolder = (holderEvents: TorchHolderEvent[]) => {
return holderEvents[holderEvents.length - 1]
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

assumes last transfer event is current holder

Copy link
Copy Markdown
Contributor

@minimalsm minimalsm left a comment

Choose a reason for hiding this comment

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

Seeing this currently

Screenshot 2025-07-21 at 14 21 55

@pettinarip
Copy link
Copy Markdown
Member Author

Seeing this currently

Screenshot 2025-07-21 at 14 21 55

expected in the preview deploy env

@pettinarip pettinarip merged commit 675798d into master Jul 21, 2025
18 checks passed
@pettinarip pettinarip deleted the hotfix-10y branch July 21, 2025 14:13
This was referenced Jul 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tooling 🔧 Changes related to tooling of the project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants