Skip to content

Conversation

mtias
Copy link
Member

@mtias mtias commented Jun 1, 2017

Be consistent about naming.

@mtias mtias added [Feature] Block API API that allows to express the block paradigm. [Type] Developer Documentation Documentation for developers Framework Issues related to broader framework topics, especially as it relates to javascript labels Jun 1, 2017
@aduth
Copy link
Member

aduth commented Jun 1, 2017

There were many more instances of "slug" terminology, updated in cf19b31. We also use the same term for block categories and block controls documentation, the latter of which is outdated and should probably be removed/updated. I can see to this.

@mtias
Copy link
Member Author

mtias commented Jun 1, 2017

Thanks, i had to flee in the middle of fixing other cases.

@mtias
Copy link
Member Author

mtias commented Jun 1, 2017

Investigating issue with inserter.

@aduth
Copy link
Member

aduth commented Jun 1, 2017

There's quite a few "blockType" occurring in the parser. Should those be updated to "blockName"?

@aduth aduth force-pushed the update/block-registration-function-docs branch from 46e3f73 to 1036a3f Compare June 14, 2017 15:32
@aduth
Copy link
Member

aduth commented Jun 14, 2017

Rebased to resolve conflicts, squash, and update parser's "block type" (string) references to "block name".

@aduth aduth requested a review from youknowriad June 14, 2017 15:33
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.

Didn't find any block slug or wrong blockType reference. 👍

@aduth aduth merged commit 38e3fdc into master Jun 14, 2017
@aduth aduth deleted the update/block-registration-function-docs branch June 14, 2017 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Block API API that allows to express the block paradigm. Framework Issues related to broader framework topics, especially as it relates to javascript [Type] Developer Documentation Documentation for developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants