fix(ipfs2filecoin): derive the $2.50 rate from USD_PER_TIB_MONTH_PER_COPY - #2
Merged
gmoranxyz merged 1 commit intoAug 5, 2026
Conversation
…COPY The rate footnote, one reason card, and the SEO description each restated $2.50 as a literal, so they could silently drift from the comparison table and estimator, which already derive from the constant.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📝 Description
Follow-up to #1, which called out this exact gap in its description: the $2.50 per TiB per month per copy rate was restated as a string literal in three places instead of being derived from
USD_PER_TIB_MONTH_PER_COPY, the same constant the comparison table and cost estimator already use. That left those three spots free to silently drift from the actual priced rate if the constant ever changed.🛠️ Key Changes
constants/seo.ts: the page's meta description now builds its rate mention fromformatUsd(USD_PER_TIB_MONTH_PER_COPY)instead of a hardcoded'$2.50'string.data/reasons.ts: the "A fraction of pinning-service pricing" reason card description is now a template literal driven by the same constant.page.tsx: the rate footnote under the comparison table now interpolates{formatUsd(USD_PER_TIB_MONTH_PER_COPY)}rather than hardcoding$2.50.All three now match exactly what
ComparisonTableandCostEstimatoralready price off of, so there's a single source of truth for the rate.📌 To-Do Before Merging
🧪 How to Test
npm install && npx next build --webpack, ornpm run devand visit/ipfs2filecoin./ipfs2filecoinand check the page's<meta name="description">tag.npx tsc --noEmitshows the same 28 pre-existing, unrelated errors on this branch with and without this change (missing static-asset type declarations elsewhere in the repo);npx biome checkis clean on the touched files;npx next build --webpackcompiles successfully and statically generates/ipfs2filecoin.📸 Screenshots
No visual change — the rendered copy is identical, only its source changed from a literal to a derived value.
🔖 Resources
🤖 Generated with Claude Code
Generated by Claude Code