Skip to content

Conversation

sansyrox
Copy link
Member

Description

This PR fixes #1077

Summary

This PR fixes the type definitions and improves the docuementation

PR Checklist

Please ensure that:

  • The PR contains a descriptive title
  • The PR contains a descriptive summary of the changes
  • You build and test your changes before submitting a PR.
  • You have added relevant documentation
  • You have added relevant tests. We prefer integration tests wherever possible

Pre-Commit Instructions:

Copy link

vercel bot commented Jan 11, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
robyn ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 11, 2025 10:19pm

@sansyrox sansyrox changed the title chore: fix the documentation [WIP] chore: fix the wrong types in the library Jan 11, 2025
from robyn.robyn import FunctionInfo, Headers, HttpMethod, Request, Response, WebSocketConnector, get_version, Url
from robyn.router import MiddlewareRouter, MiddlewareType, Router, WebSocketRouter
from robyn.types import Directory
from robyn.types import Directory, PathParams, Method, FormData, Files, IPAddress, Body
Copy link

Choose a reason for hiding this comment

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

Attempting to import types (PathParams, Method, FormData, Files, IPAddress, Body) that are not defined in robyn/types.py module based on the codebase inspection. This will cause ImportError at runtime.


React with 👍 to tell me that this comment was useful, or 👎 if not (and I'll stop posting more comments like this in the future)

Copy link

recurseml bot commented May 27, 2025

😱 Found 1 issue. Time to roll up your sleeves! 😱

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.

Missing Type Definitions: RequestMethod, RequestBody, RequestURL

1 participant