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

Documentation refers to a non-existing method MultipartWriter.serialize() #2965

Closed
vovanz opened this issue Apr 28, 2018 · 1 comment · Fixed by #2970
Closed

Documentation refers to a non-existing method MultipartWriter.serialize() #2965

vovanz opened this issue Apr 28, 2018 · 1 comment · Fixed by #2970
Labels

Comments

@vovanz
Copy link

vovanz commented Apr 28, 2018

Long story short

Working with Multipart documentation page refers to MultipartWriter.serialize() method several times.

But this method does not exist (anymore?) in the class aiohttp.MultipartWriter.

Expected behaviour

Methods mentioned in docs should actually exist :)

Actual behaviour

MultipartWriter.serialize() is mentioned in docs but does not exist.

Steps to reproduce

  1. Open the "Working with Multipart" page in docs and CTRL+F all MultipartWriter.serialize() occurrences.
  2. Open aiohttp.MultipartWriter reference and make sure that this method is not documented there.
  3. Open the source at aiohttp/multipart.py and make sure that the serialize method is not defined.

Your environment

aiohttp==3.1.3

@lock
Copy link

lock bot commented Oct 28, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a [new issue] for related bugs.
If you feel like there's important points made in this discussion, please include those exceprts into that [new issue].
[new issue]: https://github.com/aio-libs/aiohttp/issues/new

@lock lock bot added the outdated label Oct 28, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Oct 28, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant