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

Block API: Remove undefined className argument from save() methods #11605

Merged
merged 1 commit into from
Nov 8, 2018

Conversation

ocean90
Copy link
Member

@ocean90 ocean90 commented Nov 7, 2018

Description

Unlike the edit() method, save() doesn't get the className property passed to. The generated and custom classes are added through filters.

See #7308.

How has this been tested?

Unit tests and manually by checking that no blocks are invalid.

@ocean90 ocean90 added [Feature] Block API API that allows to express the block paradigm. [Type] Developer Documentation Documentation for developers [Block] Cover Affects the Cover Block - used to display content laid over a background image labels Nov 7, 2018
@ocean90 ocean90 added this to the 4.3 milestone Nov 7, 2018
Copy link
Contributor

@youknowriad youknowriad left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@ocean90 ocean90 merged commit 5728b36 into master Nov 8, 2018
@ocean90 ocean90 deleted the fix/edit-no-classname branch November 8, 2018 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Cover Affects the Cover Block - used to display content laid over a background image [Feature] Block API API that allows to express the block paradigm. [Type] Developer Documentation Documentation for developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants