Conversation
Add documentation pages for the mpp Elixir SDK (ZenHive/mpp), a community Plug middleware implementation for Phoenix and Plug apps. Closes tempoxyz#456
|
@e-fu is attempting to deploy a commit to the Tempo Team on Vercel. A member of the Team first needs to authorize it. |
There was a problem hiding this comment.
Pull request overview
Adds documentation and navigation entries for a new Elixir SDK (ZenHive/mpp) so it appears alongside existing language SDKs and is reachable from the sidebar and /sdk landing page.
Changes:
- Add
/elixirredirect and Elixir section to the SDK sidebar navigation. - Add Elixir SDK card to the
/sdklisting page. - Add new Elixir SDK docs pages: Overview, Core types, and Server (Plug middleware).
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
vocs.config.ts |
Adds /elixir redirect and sidebar nav entries for the Elixir SDK docs. |
src/pages/sdk/index.mdx |
Updates SDK listing metadata and adds the Elixir SDK card to the listing page. |
src/pages/sdk/elixir/index.mdx |
New Elixir SDK overview page with install/quickstart content and links onward. |
src/pages/sdk/elixir/core.mdx |
New page documenting core protocol/header types and helpers in the Elixir library. |
src/pages/sdk/elixir/server.mdx |
New page documenting MPP.Plug server middleware usage and configuration. |
src/components/cards.tsx |
Introduces ElixirSdkCard used by the SDK listing page. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
src/pages/sdk/index.mdx
Outdated
| # SDKs [Official implementations in multiple languages] | ||
|
|
There was a problem hiding this comment.
This page heading describes the listed SDKs as “Official implementations”, but the newly added Elixir SDK is a community-maintained project (ZenHive/mpp) per the PR description/issue. Please adjust the heading (and/or add a clear “Community” label on the Elixir card) so readers aren’t misled about support/ownership.
src/pages/sdk/index.mdx
Outdated
|
|
||
| import { Cards } from 'vocs' | ||
| import { TypeScriptSdkCard, PythonSdkCard, RustSdkCard, WalletCliCard } from '../../components/cards' | ||
| import { TypeScriptSdkCard, PythonSdkCard, RustSdkCard, ElixirSdkCard, WalletCliCard } from '../../components/cards' |
There was a problem hiding this comment.
WalletCliCard is imported here but never used in this MDX file. Please remove it from the import list to avoid unused-import lint/TS checks failing (or add the card to the <Cards> section if it was intended to be shown).
| import { TypeScriptSdkCard, PythonSdkCard, RustSdkCard, ElixirSdkCard, WalletCliCard } from '../../components/cards' | |
| import { TypeScriptSdkCard, PythonSdkCard, RustSdkCard, ElixirSdkCard } from '../../components/cards' |
- Split SDK page into Official and Community sections - Add Community badge to Elixir card - Remove unused WalletCliCard import - Add callout linking to hex.pm and hexdocs for latest version/API docs - Fix method_config keys: atom keys → string keys per actual API - Add missing network_id to Stripe config example - Add method_config to per-route pricing examples
Summary
Phoenix/Plug apps
Closes #456