Skip to content

Add new blocks to models#440

Open
yyakhnytsia wants to merge 7 commits intoHubSpot:masterfrom
yyakhnytsia:add-new-blocks-to-models
Open

Add new blocks to models#440
yyakhnytsia wants to merge 7 commits intoHubSpot:masterfrom
yyakhnytsia:add-new-blocks-to-models

Conversation

@yyakhnytsia
Copy link
Copy Markdown
Contributor

Add new block types: TaskCardBlock, MarkdownBlock and it's underlying structure to the models.

import com.hubspot.slack.client.models.blocks.elements.BlockElement;

@JsonTypeInfo(use = JsonTypeInfo.Id.NAME, property = "type")
@JsonSubTypes(
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should we also include rich_text_preformatted and rich_text_quote? What will happen if we receive a type that we do not support (if that’s somehow possible)? I am cautious about introducing logic-breaking changes.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@opaliukh added

@yyakhnytsia
Copy link
Copy Markdown
Contributor Author

@opaliukh Fixed all comments.

@yyakhnytsia yyakhnytsia requested a review from opaliukh April 16, 2026 12:44
@Value.Parameter
RichTextListStyle getStyle();

Optional<Integer> getIntend();
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
Optional<Integer> getIntend();
Optional<Integer> getIndent();

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@opaliukh fixed.

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