Skip to content

Conversation

@maxswjeon
Copy link

Hi, this is a first time opening a pull request in this project. I really appreciate your work, and I would like to provide some help if possible. I really want the custom blocks feature, so I am digging for it.

This PR is only for reference, so feel free to get some ideas and copy some codes.

The code might be not like your intentions. If so, please give short guidelines or your intensions so I can align with them.

@vercel
Copy link

vercel bot commented May 12, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
blocknote ❌ Failed (Inspect) May 12, 2023 4:09pm
blocknote-website ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 12, 2023 4:09pm

@maxswjeon
Copy link
Author

Remaining type errors (for note)

  • src/api/blockManipulation/blockManipulation.test.ts
  • src/api/formatConversions/formatConversions.test.ts
  • src/api/nodeConversions/nodeConversions.test.ts
  • src/api/nodeConversions/testUtil.ts
  • src/extensions/Blocks/api/cursorPositionTypes.ts
  • src/extensions/Blocks/api/selectionTypes.ts

@YousefED
Copy link
Collaborator

Thanks for contributing @maxswjeon, and great to see you're getting up to speed with the codebase quickly.

We're working hard on this feature as well so I think it's smart to make sure work does not overlap. Does #183 contain some of the suggestions already?

@maxswjeon
Copy link
Author

Oh, I see. I will rebase to #183 and continue working. Thanks for pointing out.

@maxswjeon maxswjeon force-pushed the custom-blocks-proposal branch from b75cd5f to 25a0911 Compare May 12, 2023 15:18
@maxswjeon maxswjeon changed the base branch from custom-blocks-proposal to custom-blocks-api-changes May 12, 2023 15:28
@maxswjeon maxswjeon changed the title Custom blocks proposal - Some ideas and chores Fix build issue on custom-blocks-api-changes May 12, 2023
@maxswjeon maxswjeon marked this pull request as ready for review May 12, 2023 15:29
@maxswjeon
Copy link
Author

This PR will fix build issue on #183. Additional discussion will be occur on #183

@YousefED
Copy link
Collaborator

superseded by #191

@YousefED YousefED closed this May 30, 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.

2 participants