Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kgoel’s picture

Assigned: Unassigned » kgoel

Assigning this to myself and will be working on this during DrupalCon lab.

Mile23’s picture

Great. :-)

kerasai’s picture

Began work on porting this example module. Still quite a ways to go.

Progress is also being pushed GitHub repo. https://github.com/kerasai/drupal-examples/tree/8.x-1.x-entity_example/e...

Mile23’s picture

Status: Active » Needs review
kgoel’s picture

Assigned: kgoel » Unassigned
Mile23’s picture

Status: Needs review » Needs work

The last submitted patch, 3: examples.entity-example.1883722-3.patch, failed testing.

Mile23’s picture

Status: Needs work » Needs review
Berdir’s picture

Issue summary: View changes

It might be easier to just start with a fresh example, there's already one for a config entity here: #2084301: Add a Config Entity example.

The other part is then adding a content entity example, we're working on something in https://github.com/christophegalli/entity_d8, which we could add for this?

Mile23’s picture

The more the merrier. I'd suggest submitting that github repo as a patch, in a new issue.

cgalli’s picture

Mile23’s picture

Priority: Normal » Minor
Status: Needs review » Postponed

Great, thanks.

Also thanks to kerasai for the first patch on this issue... I think I'll mark this one postponed, and if anyone wants to pick it up they can. Really, though, work should focus on Config and Content Entity examples for now: #2084301: Add a Config Entity example #2220565: Create a Content Entity Example

Mile23’s picture

Status: Postponed » Closed (outdated)

I think this one is outdated. It will take too much work to adapt it into the new APIs, and we already have content_entity_example and config_entity_example.

Thanks!