Feature/allowed-blocks

[dlousa92]

No description provided.

[jnschrag]

Thanks for taking a first pass at this - it's very much appreciated!

• Can you check your indentation? All items in the array should have the same indentation.
• If you could alphabetize these, that would be great! It'll make it easier on us as we adjust this list/need to easily scan them.
• Let's add the following items:
  • core/table
  • core/html
  • core/video
  • core/embed
  • core-embed/twitter
  • core-embed/youtube
  • core-embed/facebook
  • core-embed/soundcloud
  • core-embed/flickr
  • core-embed/vimeo
• Does the whitelist affect the blocks that are added by plugins? If it does, then we also need to make sure that the Jetpack slideshow & tiled gallery blocks are included

[jnschrag]

Can you add a blank line at the end of the file? You also don't need the closing ?>

[dlousa92]

I went ahead and added the additional blocks and reordered them alphabetically. Also fixed indentation.

As far as plugins being affected. It looks like the whitelist does affect those. I was trying to find what the jetpack slugs were called so I could add them to the list, but I haven't had any luck.

[jnschrag]

@dlousa92 Thanks for making these improvements.

Re your comment about not being able to find Jetpack's blocks, you can take a look through their /blocks/ directory here: https://github.com/Automattic/jetpack/search?q=jetpack_register_block&unscoped_q=jetpack_register_block If you look at where the jetpack_register_block function is being called, you should be able to get the name of the ones we want.

[dlousa92]

Thank you! So I have the correct names now, however they're not all popping up in the editor. 'jetpack/block-name' does work, as can be seen by the example 'jetpack/slideshow' I included in the code. However 'jetpack/tiled-gallery' or 'jetpack/carousel' do not show up.