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: sync PageMeta.title with PageBlockModel.title while importing markdown files #5297

Merged
merged 4 commits into from
Nov 12, 2023

Conversation

hyf0
Copy link
Contributor

@hyf0 hyf0 commented Nov 12, 2023

Fixes #5296. Thank @lawvs for guidance.

Copy link

vercel bot commented Nov 12, 2023

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

Name Status Preview Comments Updated (UTC)
blocksuite ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 12, 2023 5:55am
1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
blocksuite-docs ⬜️ Ignored (Inspect) Visit Preview Nov 12, 2023 5:55am

Copy link

@affine-bot affine-bot left a comment

Choose a reason for hiding this comment

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

Hello! Thank you for your contribution.

In order for us to review and merge your suggested changes, please sign our CLA at https://github.com/toeverything/blocksuite/edit/master/.github/CLA.md

@fourdim
Copy link
Contributor

fourdim commented Nov 12, 2023

Hi, could you please open a new pull request to add your name directly to the https://github.com/toeverything/blocksuite/edit/master/.github/CLA.md so that we can review this PR?

@hyf0
Copy link
Contributor Author

hyf0 commented Nov 12, 2023

Hi, could you please open a new pull request to add your name directly to the https://github.com/toeverything/blocksuite/edit/master/.github/CLA.md so that we can review this PR?

Sure. #5298

@fourdim
Copy link
Contributor

fourdim commented Nov 12, 2023

This PR will be approved and merged once the CLA PR is approved.

Copy link
Member

@lawvs lawvs left a comment

Choose a reason for hiding this comment

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

It works, thank you!

Screen.Recording.2023-11-12.at.14.05.25.mov

@fourdim fourdim merged commit f6c23e8 into toeverything:master Nov 12, 2023
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Archived in project
Development

Successfully merging this pull request may close these issues.

Importing markdown files doesn't update the name in PageMeta and WorkspaceMeta correctly
4 participants