Skip to content

Update documentation about plans#1759

Open
Viicos wants to merge 1 commit intomainfrom
vp/plans-docs
Open

Update documentation about plans#1759
Viicos wants to merge 1 commit intomainfrom
vp/plans-docs

Conversation

@Viicos
Copy link
Member

@Viicos Viicos commented Mar 6, 2026

No description provided.

@Viicos Viicos requested a review from alexmojaki March 6, 2026 15:11
@cloudflare-workers-and-pages
Copy link

Deploying logfire-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: a4df4c6
Status: ✅  Deploy successful!
Preview URL: https://f0e4da93.logfire-docs.pages.dev
Branch Preview URL: https://vp-plans-docs.logfire-docs.pages.dev

View logs

Copy link
Contributor

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

Choose a reason for hiding this comment

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

Devin Review found 1 potential issue.

Open in Devin Review

Copy link
Contributor

Choose a reason for hiding this comment

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

🚩 Stale "Pro" references remain in comparison docs outside this PR

While this PR correctly updates references to "Pro plan"/"Pro rate"/"Pro tier" in the three modified files, there are still two references to "Pro" in other documentation files that were not updated:

  • docs/comparisons/langsmith.md:24: *Logfire Cloud Pro pricing. Enterprise pricing available on request.
  • docs/comparisons/datadog.md:29: *Logfire Cloud Pro pricing (Team or Growth plans -> $2/million spans).

The datadog.md reference is particularly interesting because it already mentions "Team or Growth plans" in parentheses but still uses the "Pro" label. The langsmith.md reference has no such qualification.

These are pre-existing and outside the PR's diff, but since the intent of this PR is to remove "Pro" terminology, these should ideally be updated in the same PR for consistency.

Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

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.

1 participant