Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cs_shadow’s picture

Status: Active » Postponed
Issue tags: +revisit before release

Postponing until we freeze features of the module, otherwise this'll also need to be updated with every PR.

Wim Leers’s picture

Status: Postponed » Active
Issue tags: - +revisit before release

d.o somehow ate this tag that @cs_shadow added in #1… strange!

Anyway, now that we're getting closer to a release, it probably makes sense to actually do this. I'll take a stab at it later this week, after I've gone through all of Entity Embed.

Wim Leers’s picture

Status: Active » Needs review
Issue tags: -revisit before release
FileSize
1.52 KB

Anyway, now that we're getting closer to a release, it probably makes sense to actually do this. I'll take a stab at it later this week, after I've gone through all of Entity Embed.

… I didn't end up having time back then to continue going through all of Entity Embed, but as irony would have it, I am right now! 😅😂

So, about 3 years late, here it is!

Status: Needs review » Needs work

The last submitted patch, 3: 2309301-3.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

Wim Leers’s picture

Status: Needs work » Needs review
phenaproxima’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: +Documentation

I like it!

phenaproxima’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.