-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Activity editor add File block #3146
Activity editor add File block #3146
Conversation
Oh this is super cool! |
I tried to upload a file but I only see a request |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can't get upload to work when I test, do I need to enable some feature flag ?
The upload method was behind the image block feature flag - it's fixed now :) |
@lucasbordeau Leaving this one to you as you are working on this part |
|
||
import { FileBlock } from './FileBlock'; | ||
|
||
export const blockSpecs: any = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it possible to have a type here ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Im not sure about the types on this one - it's kind of weird what is expected by the Blockeditor generics. Maybe you or someone else has an idea how to do this the right way :)
Bildschirmaufnahme.2023-12-23.um.13.38.36.mov
For now it works but its not quite production ready :)
But I think with a bit of work on cmd+c/v and drop events this could work really well - maybe we also want to change the design of the block or convert it to an inline content?