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

implemented chat feature using sidecar from starlette docs#1061

Merged
tiangolo merged 4 commits intofastapi:masterfrom
aakashnand:chat_in_docs
Mar 27, 2020
Merged

implemented chat feature using sidecar from starlette docs#1061
tiangolo merged 4 commits intofastapi:masterfrom
aakashnand:chat_in_docs

Conversation

@aakashnand
Copy link
Contributor

@aakashnandaakashnand commentedMar 1, 2020
edited
Loading

As@tiangolo mentioned in#1001. I thought of implementing chat in the documentation. This is my first PR for Fastapi and on github. Let me know if I miss something. I have tested locally and it is working fine. Below are some screenshots.

  • Open chat button at right bottom

Screen Shot 2020-03-01 at 3 46 26 PM

  • Chat sidecar from fastapigitter room

Screen Shot 2020-03-01 at 3 48 25 PM

aviramha reacted with thumbs up emoji
@codecov
Copy link

codecovbot commentedMar 1, 2020
edited
Loading

Codecov Report

Merging#1061 intomaster willnot change coverage by%.
The diff coverage isn/a.

Impacted file tree graph

@@            Coverage Diff            @@##            master     #1061   +/-   ##=========================================  Coverage   100.00%   100.00%           =========================================  Files          222       222             Lines         6569      6569           =========================================  Hits          6569      6569

Continue to review full report at Codecov.

Legend -Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing data
Powered byCodecov. Last update433d786...8400565. Read thecomment docs.

@tiangolotiangolo merged commit2738df3 intofastapi:masterMar 27, 2020
@tiangolo
Copy link
Member

Great! Thank you! 🍰 🚀

And thanks for taking the effort to take screenshots 👌

aakashnand reacted with thumbs up emoji

@aakashnand
Copy link
ContributorAuthor

@tiangolo Its Github's PR feature anniversary today. I checked my first PR usinghttps://firstpr.me/ and it turned out to be this PR.

What an absolute pleasure to start my open-source journey with you@tiangolo

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@aakashnand@tiangolo

Comments


[8]ページ先頭

©2009-2026 Movatter.jp