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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃摑 Remove note about (now supported) feature from Swagger UI in docs/en/docs/tutorial/request-files.md #2803

Merged
merged 3 commits into from Oct 7, 2021

Conversation

gsganden
Copy link
Contributor

It appears to me that this note is no longer needed because Swagger UI now does support multiple file uploads.

Screen Shot 2021-02-12 at 4 39 34 PM

Remove outdated note about Swagger UI
@codecov
Copy link

codecov bot commented Feb 12, 2021

Codecov Report

Merging #2803 (007ee5b) into master (426bd09) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master     #2803   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          409       409           
  Lines        10264     10264           
=========================================
  Hits         10264     10264           

Continue to review full report at Codecov.

Legend - Click here to learn more
螖 = absolute <relative> (impact), 酶 = not affected, ? = missing data
Powered by Codecov. Last update 426bd09...007ee5b. Read the comment docs.

@github-actions
Copy link
Contributor

馃摑 Docs preview for commit 22cf10e at: https://602704a4a497080d5d7e9224--fastapi.netlify.app

Copy link

@JonasKs JonasKs left a comment

Choose a reason for hiding this comment

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

Agree, just did the tutorial and noticed this.

@tiangolo tiangolo changed the title Update request-files.md 馃摑 Remove note about (now supported) feature from Swagger UI in docs/en/docs/tutorial/request-files.md Oct 7, 2021
@tiangolo
Copy link
Owner

tiangolo commented Oct 7, 2021

Awesome, thanks @gsganden! 馃殌 馃嵃

@tiangolo tiangolo enabled auto-merge (squash) October 7, 2021 12:48
@github-actions
Copy link
Contributor

github-actions bot commented Oct 7, 2021

馃摑 Docs preview for commit 007ee5b at: https://615eee1c53eb8b1d1ff29d60--fastapi.netlify.app

@tiangolo tiangolo merged commit afe1f69 into tiangolo:master Oct 7, 2021
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.

None yet

3 participants