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

forward args to flask from wsgi #573

Merged

Conversation

RedX2501
Copy link
Contributor

To enable accessing the backend on the local network add the possibility of passing host to the wsgi script.

Additionally improve the documentation to show these new features and clarify how to debug.

@TomBursch
Copy link
Owner

Again, thanks for this and helping others in the future!
Initially, I didn't want to add any information about any IDE, but I think you're right, it's better to have that.

I only have some minor things:

  • I would suggest opening the backend folder and the frontend/kichenowl folder in separate VSCode windows, this makes it a bit easier to work with the project.
  • Remove the cwd and args from the vscode configuration
  • And add this flutter vscode configuration:
{
    "name": "kitchenowl",
    "request": "launch",
    "type": "dart"
},
{
    "name": "kitchenowl (profile mode)",
    "request": "launch",
    "type": "dart",
    "flutterMode": "profile"
}

@RedX2501
Copy link
Contributor Author

Yeah i haven't added those as they are automatically suggested by VS code.

@RedX2501 RedX2501 force-pushed the feature/forward-args-to-flask-from-wsgi branch from e718bf3 to 8dccf8e Compare December 13, 2024 09:15
@TomBursch TomBursch added the documentation Improvements or additions to documentation label Dec 13, 2024
@TomBursch TomBursch merged commit 4fa523c into TomBursch:main Dec 13, 2024
1 check passed
@RedX2501 RedX2501 deleted the feature/forward-args-to-flask-from-wsgi branch December 13, 2024 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants