Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add FastHTMLPlugin (partial checkin) #10

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Conversation

audreyfeldroy
Copy link
Member

The missing piece I think is to mock the request when I call get() for a route

The missing piece I think is to mock the request when I call get() for a route
@audreyfeldroy audreyfeldroy requested a review from pydanny August 6, 2024 20:16
…L files for each route

* **Error Handling**
  - Add error handling for missing `app` or `rt` attributes in the module
  - Continue processing other files if `app` or `rt` is missing

* **Route Execution**
  - Execute only GET routes to generate HTML
  - Save generated HTML files with route-specific names

Add test case in `test_main.py` to verify FastHTML plugin generates HTML files correctly

* **Setup and Cleanup**
  - Create temporary input directory with a FastHTML app
  - Remove temporary directories after test

* **Verification**
  - Check if HTML file is generated correctly
  - Verify content of generated HTML file

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/feldroy/air/pull/10?shareId=XXXX-XXXX-XXXX-XXXX).
@audreyfeldroy
Copy link
Member Author

Update run method in fasthtml_plugin.py to correctly generate HTML files for each route

  • Error Handling

    • Add error handling for missing app or rt attributes in the module
    • Continue processing other files if app or rt is missing
  • Route Execution

    • Execute only GET routes to generate HTML
    • Save generated HTML files with route-specific names

Add test case in test_main.py to verify FastHTML plugin generates HTML files correctly

  • Setup and Cleanup

    • Create temporary input directory with a FastHTML app
    • Remove temporary directories after test
  • Verification

    • Check if HTML file is generated correctly
    • Verify content of generated HTML file

For more details, open the Copilot Workspace session.

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