docs: address BigLep review feedback (GER-1011)#77
Merged
Conversation
Resolve the high/medium items from BigLep's review of filecoin-project#2457: fix broken/missing redirects, correct stale entry-point copy, and remove the misleading moved-content page. - Delete reference/general/moved-content.md (Filecoin Pin lives in-repo) and drop it from SUMMARY.md - Add redirects for builder-cookbook/data-storage/{store-data, retrieve-data,privacy-and-access-control} - Repoint builder-cookbook/dapps/{oracles,cross-chain-bridges} to build/advanced/*; fix smart-contracts/fundamentals/overview mapping - Archive the chain-data-query tutorial in legacy-content.md and repoint its redirect - Update WELCOME.md cards to the new IA labels and links - Align cookbook README with the reference-only/link-out strategy - Replace hardcoded FOC pricing with links to the FOC docs - Add a scheduled check:rpcs CI workflow - Fix stale "Was this page helpful?" prefill URLs in the cookbook Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
✅ Deploy Preview for timely-kataifi-9f3a09 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Add Curio (with links to curiostorage.org and docs.curiostorage.org) as the modern storage-provider stack in the direct deal-making section, so the builder-facing decision page no longer reads as Boost/Lotus-only. Addresses BigLep review item 15.
nijoe1
force-pushed
the
ger-1011-biglep-review-fixes
branch
from
9e159a3 to
1b7b933
Compare
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
1 participant
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.
Addresses GER-1011 — BigLep's review feedback on filecoin-project#2457. Stacks on top of
upstream/docs-ia-refresh.✅ Fixed in this PR
Redirects & links
reference/general/moved-content.mdand removed it fromSUMMARY.md— Filecoin Pin docs live in-repo atbuild/cookbook/filecoin-pin/, so the "moved to filecoin.cloud" page was misleading. (item 1)builder-cookbook/data-storage/{store-data,retrieve-data,privacy-and-access-control}→ their new homes. (item 2)builder-cookbook/dapps/oraclesand…/cross-chain-bridges→build/advanced/*(were going to the cookbook index). (item 3)smart-contracts/fundamentals/overview(was mapped to a node-implementations page) →build/getting-started.md. (item 4, overview half)chain-data-querytutorial inlegacy-content.mdand repointed its redirect there instead of the cookbook index. (item 10)Copy & accuracy
2.5 USDFC/TiB/monthand egress figures with links to the FOC docs. (item 8)upload-to-filecoin.mddirect deal-making now leads with Curio (docs) as the modern SP stack, with Boost/Lotus as context, so the builder-facing page no longer reads as Boost/Lotus-only. (item 15)CI
check:rpcsworkflow (weekly + manual) so the RPC tables don't drift silently. (item 9)Validation:
npm run buildpasses (no errors).⏸️ Deferred (need a decision)
smart-contracts/fundamentals/roadmap→ FVM README still works, but the roadmap timeline content was deleted with no archive. Left as-is pending a call: accept FVM README as the landing, archive a historical stub, or point to a live roadmap.retrieve-data.mdstill recommends Lassie withlassie/pkg/...imports; not yet verified against the current module layout.💬 For BigLep (on filecoin-project#2457)
docs-ia-refresh— top-levelsmart-contracts/programmatic-storagealready redirects toreference/general/legacy-content.md(your recommended target), so no change was needed.upload-to-filecoin.mdso both surfaces are aligned.