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

[DEV-7540] Implement type guards for Title/Subtitle headings #22

Merged

Conversation

kudlajz
Copy link
Contributor

@kudlajz kudlajz commented Jun 6, 2023

No description provided.

@kudlajz kudlajz requested a review from e1himself June 6, 2023 09:07
@kudlajz kudlajz self-assigned this Jun 6, 2023
@@ -26,6 +33,22 @@ export namespace HeadingNode {
return ComposedElement.isComposedElement(value, type);
}

export function isTitleHeadingNode<Heading extends HeadingNode<Child>, Child extends Node>(
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Unfortunately I wasn't able to test it in the content renderer package, because linking the @prezly/story-content-format module worked, but it wouldn't resolve the underlying imports from @prezly/content-format 😕

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've checked that the typeguards still worked, just wasn't able to compile it due to the mentioned import not being resolved.

@kudlajz kudlajz merged commit a50b1d7 into main Jun 6, 2023
3 checks passed
@kudlajz kudlajz deleted the feature/dev-7540-update-prezlycontent-renderer-react-js branch June 6, 2023 11:03
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.

None yet

2 participants