Skip to content

Update PRAW migration guide#113

Open
lilspazmin wants to merge 1 commit into
mainfrom
praw-migration
Open

Update PRAW migration guide#113
lilspazmin wants to merge 1 commit into
mainfrom
praw-migration

Conversation

@lilspazmin
Copy link
Copy Markdown

@lilspazmin lilspazmin commented May 21, 2026

💸 TL;DR

Updates and fleshes out the docs for PRAW → Devvit migration guide.

📜 Details

🧪 Testing Steps / Validation

Code samples verified manually in a test Devvit app.

✅ Checks

  • CI tests (if present) are passing
  • Adheres to code style for repo
  • Contributor License Agreement (CLA) completed if not a Reddit employee

@lilspazmin lilspazmin requested a review from a team May 21, 2026 21:34
Comment thread docs/guides/migrate/public-api.md Outdated
return c.json<TriggerResponse>({ status: 'ok' }, 200);
app.post('/internal/triggers/on-comment-create', async (c) => {
const { subredditName } = context;
const input = await c.req.json<{ comment?: { id?: string; body?: string; postId?: string } }>();
Copy link
Copy Markdown

@fsvreddit fsvreddit May 22, 2026

Choose a reason for hiding this comment

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

Wouldn't this be clearer?

const input = await c.req.json<OnCommentCreateRequest>();

With the corresponding import on examples on this page. Having the full interfaces available make it a lot easier to figure out what's available rather than flailing around and making guesses.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants