Skip to content
This repository has been archived by the owner on Jan 5, 2018. It is now read-only.

Ckeditor integration #11

Merged
merged 1 commit into from Aug 13, 2015

Conversation

prateekmehta
Copy link
Contributor

Includes Preview Test

@cs-shadow
Copy link
Member

EDIT: I also checked it - Admin tests are passing locally

@cs-shadow
Copy link
Member

Cool, the tests are all passing now. Weird why the AdminTest failed earlier.

}
}

//var entity_label = data.label ? data.label : existingValues['data-entity-label'];
Copy link
Member

Choose a reason for hiding this comment

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

remove this stray comment

@cs-shadow
Copy link
Member

I don't know whats up with Travis, its showing random errors on builds and the errors change when I rebuild it.

@prateekmehta
Copy link
Contributor Author

@davereid Could you also please review it, so i can move forward with filed formatter.

@davereid
Copy link
Member

@prateekmehta The field formatter can be done completely independently of any of this work, please do not let it hold you up.

@davereid
Copy link
Member

It would be nice to get a rebase on all of these commits.

@cs-shadow
Copy link
Member

@davereid Seems like they already are rebased.

@davereid
Copy link
Member

Grr, Github not updating. Thanks, I see that now.

davereid added a commit that referenced this pull request Aug 13, 2015
@davereid davereid merged commit b02b478 into drupal-media:8.x-1.x Aug 13, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants