Skip to content

Fix build issue on custom-blocks-api-changes #192

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

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