Skip to content

docs(wallets): stop emitting the V1 version banner in generated SDK reference docs - #2042

Merged
albertoelias-crossmint merged 1 commit into
mainfrom
devin/1788201932-remove-wallets-v1-docs-banner
Aug 31, 2026
Merged

docs(wallets): stop emitting the V1 version banner in generated SDK reference docs#2042
albertoelias-crossmint merged 1 commit into
mainfrom
devin/1788201932-remove-wallets-v1-docs-banner

Conversation

@albertoelias-crossmint

@albertoelias-crossmint albertoelias-crossmint commented Aug 31, 2026

Copy link
Copy Markdown
Collaborator

Description

The "This page has been updated for Wallets SDK V1… see the previous version docs or the V1 migration guide" <Note> banner is being removed from docs.crossmint.com. The sdk-reference/wallets/** pages are generated from this repo, so the banner has to stop being emitted here or the next sdk-reference sync would put it back.

  • packages/wallets/typedoc-custom-plugin.mjs: drop versionBanner() and the block that prepended it after the frontmatter (typescript reference pages).
  • packages/client/ui/react-ui and react-native generators: drop the versionBanner product config; the now-unused emitBanner() and its call sites are removed from the shared engine.

Companion docs PR removing the already-generated banner: Paella-Labs/crossbit-main#29528.

Test plan

  • Generator-only change with no runtime code touched; verified no versionBanner/emitBanner references remain and biome reports no new diagnostics on the four touched files.

Package updates

  • Patch changeset for @crossmint/wallets-sdk, @crossmint/client-sdk-react-ui, @crossmint/client-sdk-react-native-ui (doc generation only).

Link to Devin session: https://crossmint.devinenterprise.com/sessions/b56772d84fdb4fc4b9570779bae1d81a
Open in Devin Desktop: https://crossmint.devinenterprise.com/desktop/session/b56772d84fdb4fc4b9570779bae1d81a?variant=devin
Requested by: @albertoelias-crossmint


Devin Review

…eference docs

Co-Authored-By: Alberto Elias <alberto.elias@paella.dev>
@devin-ai-integration

Copy link
Copy Markdown
Contributor
Original prompt from Alberto Elias

lets remove all mentions of:

"This page has been updated for Wallets SDK V1. If you are using the previous version, see the previous version docs or the V1 migration guide."

From the docs

@devin-ai-integration

Copy link
Copy Markdown
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR that start with 'DevinAI' or '@devin'.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@changeset-bot

changeset-bot Bot commented Aug 31, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: aebb146

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

This PR includes changesets to release 9 packages
Name Type
@crossmint/client-sdk-react-native-ui Patch
@crossmint/client-sdk-react-ui Patch
@crossmint/wallets-sdk Patch
@crossmint/wallets-playground-expo Patch
@crossmint/auth-ssr-nextjs-demo Patch
@crossmint/client-sdk-nextjs-starter Patch
@crossmint/wallets-quickstart-devkit Patch
@crossmint/wallets-playground-react Patch
@crossmint/client-sdk-react-base 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

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

@greptile-apps

greptile-apps Bot commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Reviews (1): Last reviewed commit: "docs(wallets): stop emitting the V1 vers..." | Re-trigger Greptile

@jmderby jmderby left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We don't technically need a release here but i don't think it will matter either way. whatever you prefer :-)

@albertoelias-crossmint
albertoelias-crossmint merged commit 90507b0 into main Aug 31, 2026
5 of 6 checks passed
@albertoelias-crossmint
albertoelias-crossmint deleted the devin/1788201932-remove-wallets-v1-docs-banner branch August 31, 2026 22:06
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