Skip to content
This repository was archived by the owner on Jul 11, 2024. It is now read-only.

Support atomic block without entity - #53

Open
Doss wants to merge 1 commit into
masterfrom
atomic-block-without-entity
Open

Support atomic block without entity#53
Doss wants to merge 1 commit into
masterfrom
atomic-block-without-entity

Conversation

@Doss

@Doss Doss commented Mar 30, 2017

Copy link
Copy Markdown

No description provided.

@oltodo
oltodo temporarily deployed to drinkjs-pr-53 March 30, 2017 12:38 Inactive

@TomRibuot TomRibuot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

On est vraiment sur d'autoriser la création d'un bloc embed sans entité associée?
On devrait pas plutôt annuler sa création si aucun paramètre n'est fourni?
Sinon ça risque de nous laisser des blocs invisibles dans l'éditeur et non modifiables du coup

@Doss

Doss commented Mar 30, 2017

Copy link
Copy Markdown
Author

Rien ne l'empêche.
Et dans me cas, pour je ne sais quel raison, drink a crée un block atomic vide dans le state et à cause de ça il était impossible d'ouvrir par le suite ce state enregistré en base.
En plus une PR similaire est en cours sur le DraftBundle -> BedrockStreaming/DraftjsBundle#7

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants