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: update slack image upload to use new API #256

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

moustafab
Copy link

The old files.upload API is deprecated and will be removed soon. It doesn't work for new slack workspaces created after May 2024.

The new flow requires a 3 step process:

  1. Call files.getUploadURLExternal to get the upload URL and file ID.
  2. Upload the file to the URL using a PUT request.
  3. Call files.completeUploadExternal to complete the upload and link it to the thread.

Addresses #255, #235, and grafana/grafana#93398

@CLAassistant
Copy link

CLAassistant commented Nov 27, 2024

CLA assistant check
All committers have signed the CLA.

@moustafab moustafab force-pushed the moustafab/fix-slack-image-upload branch 3 times, most recently from be2db7f to 2d1e4e6 Compare November 29, 2024 17:15
@moustafab moustafab self-assigned this Nov 29, 2024
The old `files.upload` API is deprecated and will be removed soon. It doesn't work for new slack workspaces created after May 2024.

The new flow requires a 3 step process:
1. Call `files.getUploadURLExternal` to get the upload URL and file ID.
2. Upload the file to the URL using a PUT request.
3. Call `files.completeUploadExternal` to complete the upload and link it to the thread.

Addresses #255, #235, and grafana/grafana#93398
@moustafab moustafab force-pushed the moustafab/fix-slack-image-upload branch from 2d1e4e6 to 0512488 Compare November 29, 2024 17:25
@moustafab moustafab marked this pull request as ready for review November 29, 2024 17:27
@moustafab moustafab requested a review from a team as a code owner November 29, 2024 17:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In review
Development

Successfully merging this pull request may close these issues.

2 participants