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

fix: images sending as 0 bytes in paginator if sent as io.BytesIO #1881

Merged
merged 11 commits into from Jan 18, 2023
Merged

fix: images sending as 0 bytes in paginator if sent as io.BytesIO #1881

merged 11 commits into from Jan 18, 2023

Conversation

CompuGenius-Programs
Copy link
Contributor

Summary

This fixes a bug where if a page is viewed more than once, the image wouldn't be displayed except the first time.

Information

  • This PR fixes an issue.
  • This PR adds something new (e.g. new method or parameters).
  • This PR is a breaking change (e.g. methods or parameters removed/renamed).
  • This PR is not a code change (e.g. documentation, README, typehinting,
    examples, ...).

Checklist

  • I have searched the open pull requests for duplicates.
  • If code changes were made then they have been tested.
    • I have updated the documentation to reflect the changes.
  • If type: ignore comments were used, a comment is also left explaining why.

This fixes a bug where if a page is viewed more than once, the image wouldn't be displayed except the first time.

Signed-off-by: Yaakov Schlachter <contact@cgprograms.com>
Signed-off-by: Yaakov Schlachter <contact@cgprograms.com>
@CompuGenius-Programs CompuGenius-Programs requested a review from a team as a code owner January 17, 2023 17:57
@Lulalaby
Copy link
Member

Lulalaby commented Jan 17, 2023

Please stick to to the commit conventions

Signed-off-by: Yaakov Schlachter <contact@cgprograms.com>
@CompuGenius-Programs CompuGenius-Programs changed the title Fixed images sending as 0 bytes in paginator fix: images sending as 0 bytes in paginator Jan 17, 2023
@CompuGenius-Programs
Copy link
Contributor Author

Please stick to to the commit conventions

I apologize, fixed the PR title. Unfortunately, can't change the commit titles of the individual commits.

@Lulalaby
Copy link
Member

Please stick to to the commit conventions

I apologize, fixed the PR title. Unfortunately, can't change the commit titles of the individual commits.

That's fine, we squash per default, just the pr title is important

@Lulalaby Lulalaby enabled auto-merge (squash) January 17, 2023 18:13
CHANGELOG.md Outdated Show resolved Hide resolved
auto-merge was automatically disabled January 17, 2023 18:15

Head branch was pushed to by a user without write access

Co-authored-by: Lala Sabathil <aiko@aitsys.dev>
Signed-off-by: Yaakov Schlachter <contact@cgprograms.com>
CHANGELOG.md Outdated Show resolved Hide resolved
discord/ext/pages/pagination.py Outdated Show resolved Hide resolved
@Lulalaby Lulalaby enabled auto-merge (squash) January 17, 2023 18:21
@CompuGenius-Programs CompuGenius-Programs changed the title fix: images sending as 0 bytes in paginator fix: images sending as 0 bytes in paginator if sent as io.BytesIO Jan 17, 2023
@CompuGenius-Programs
Copy link
Contributor Author

Currently working with @BobDotCom to actually get the best fix possible across the whole framework, instead of a local fix to this function. My code will be deprecated then.

plun1331
plun1331 previously approved these changes Jan 17, 2023
CHANGELOG.md Outdated Show resolved Hide resolved
@Lulalaby
Copy link
Member

does this works now @BobDotCom

@BobDotCom
Copy link
Member

does this works now @BobDotCom

Apart from my suggested change, this has been tested and should be ready.

Co-authored-by: plun1331 <49261529+plun1331@users.noreply.github.com>
Signed-off-by: Yaakov Schlachter <contact@cgprograms.com>
Signed-off-by: BobDotCom <71356958+BobDotCom@users.noreply.github.com>
@Lulalaby Lulalaby enabled auto-merge (squash) January 18, 2023 17:22
@Lulalaby Lulalaby merged commit 3941b31 into Pycord-Development:master Jan 18, 2023
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

4 participants