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

Support sending multiple files to paste service at once #188

Merged
merged 4 commits into from
Jul 20, 2023

Conversation

ChrisLovering
Copy link
Member

@ChrisLovering ChrisLovering commented Jul 11, 2023

This will be a breaking change, that will be released alongside #184 in v10

Changelog:
image
models:
image

@netlify
Copy link

netlify bot commented Jul 11, 2023

Deploy Preview for bot-core ready!

Name Link
🔨 Latest commit d087bec
🔍 Latest deploy log https://app.netlify.com/sites/bot-core/deploys/64b8f941a430280008dea542
😎 Deploy Preview https://deploy-preview-188--bot-core.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@ChrisLovering ChrisLovering force-pushed the paste-servic-emultiple-files branch 4 times, most recently from 702fd9b to b30c974 Compare July 14, 2023 17:38
@ChrisLovering ChrisLovering force-pushed the paste-servic-emultiple-files branch from b30c974 to e76052f Compare July 14, 2023 17:40
Copy link
Member

@shtlrs shtlrs left a comment

Choose a reason for hiding this comment

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

Looks & works great.

I guess that for the upload error thing, we can just handle that better client-side if it ever annoys us.

pydis_core/utils/paste_service.py Outdated Show resolved Hide resolved
pydis_core/utils/paste_service.py Show resolved Hide resolved
@ChrisLovering ChrisLovering merged commit b52f1ab into main Jul 20, 2023
@ChrisLovering ChrisLovering deleted the paste-servic-emultiple-files branch July 20, 2023 09:51
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.

2 participants