Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Comments

📝 Update Deta deployment tutorial for compatibility with Deta Space#6004

Merged
tiangolo merged 23 commits intofastapi:masterfrom
sudo-subg:master
May 8, 2023
Merged

📝 Update Deta deployment tutorial for compatibility with Deta Space#6004
tiangolo merged 23 commits intofastapi:masterfrom
sudo-subg:master

Conversation

@sudo-subg
Copy link
Contributor

No description provided.

KensukeOta reacted with rocket emoji
@github-actions
Copy link
Contributor

📝 Docs preview for commit0b1015d at:https://63ed3437f56a5a0b8b5107a3--fastapi.netlify.app

@sudo-subg
Copy link
ContributorAuthor

WIP / draft

@Ryandaydev
Copy link

Are there any tests associated with this PR?

@sudo-subg
Copy link
ContributorAuthor

sudo-subg commentedFeb 21, 2023
edited
Loading

Are there any tests associated with this PR?
@Ryandaydev, no. Just an update on how to deploy a FastAPI app on Deta for compatibility with Space


@app.get("/items/{item_id}")
def read_item(item_id: int):
print({"item_id": item_id})

Choose a reason for hiding this comment

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

Suggested change
print({"item_id": item_id})
print({"item_id": item_id})

@github-actions
Copy link
Contributor

📝 Docs preview for commit1338fc5 at:https://63f78a8c1643cf265e41d8e8--fastapi.netlify.app

@app.get("/items/{item_id}")
def read_item(item_id: int):
async def read_item(item_id: int):
print({"item_id": item_id})

Choose a reason for hiding this comment

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

would remove

@github-actions
Copy link
Contributor

📝 Docs preview for commitc6052d1 at:https://63fe05f5e66d1b1476d2cae2--fastapi.netlify.app

@github-actions
Copy link
Contributor

📝 Docs preview for commitaaa48cf at:https://63fe1fd1421e2c0d783763a7--fastapi.netlify.app

@github-actions
Copy link
Contributor

📝 Docs preview for commit904b621 at:https://63fe41853295e9147523ff47--fastapi.netlify.app

@github-actions
Copy link
Contributor

📝 Docs preview for commit6cafe99 at:https://63fe739d3295e93fb3240118--fastapi.netlify.app

@github-actions
Copy link
Contributor

📝 Docs preview for commitfdb61b9 at:https://63fefcb7d08d5446ade7f581--fastapi.netlify.app

@github-actions
Copy link
Contributor

📝 Docs preview for commit0d65d03 at:https://641aea09031a44054a70c337--fastapi.netlify.app

@github-actions
Copy link
Contributor

📝 Docs preview for commit1287683 at:https://643955486ac93f00770378d2--fastapi.netlify.app

@tiangolotiangolo changed the titleUpdating the Deta deployment tutorial for compatibility with Deta Space📝 Update Deta deployment tutorial for compatibility with Deta SpaceMay 8, 2023
@github-actions
Copy link
Contributor

📝 Docs preview for commit6bd7fe2 at:https://6458e3861db33321daa14ef6--fastapi.netlify.app

@github-actions
Copy link
Contributor

📝 Docs preview for commit4216a16 at:https://6458ea680554c8318143eb7d--fastapi.netlify.app

@tiangolo
Copy link
Member

tiangolo commentedMay 8, 2023
edited
Loading

Thanks@mikBighne98! 🍰

I tweaked the docs a bit, fixed a couple of things, updated a couple others for consistency, renamed the images to keep the old ones for the translations, and fixed the console outputs, this is ready! 🚀

sudo-subg reacted with heart emoji

@tiangolotiangolo merged commit724060d intofastapi:masterMay 8, 2023
@sudo-subg
Copy link
ContributorAuthor

Thank you for reviewing and merging the PR,@tiangolo 🙂

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

Reviewers

2 more reviewers

@santigandolfosantigandolfosantigandolfo left review comments

@abdelhaiabdelhaiabdelhai requested changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@sudo-subg@Ryandaydev@tiangolo@abdelhai@santigandolfo@lemonyte@xeust

[8]ページ先頭

©2009-2026 Movatter.jp