1. The module description is very technical and pretty scary: Allows entities to be rendered using a text format.
    But Embed talks about text editors, not text formats. This module should be consistent with that.
    Also, it's not about rendering, but embedding. "Rendering" sounds foreign to non-technical users.
  2. Everything from this point on is about the README.
  3. It still refers to 8.x dev releases
  4. It still just says to whitelist <drupal-entity>, which is incorrect because it's not enough: we need to whitelist the necessary attributes also.
  5. #2471579: Update README.md introduced numerous language (and logical) errors.
CommentFileSizeAuthor
#2 2628150-2.patch4.95 KBwim leers

Comments

Wim Leers created an issue. See original summary.

wim leers’s picture

Assigned: wim leers » Unassigned
Status: Active » Needs review
StatusFileSize
new4.95 KB
slashrsm’s picture

Status: Needs review » Fixed
Issue tags: +Zurich media sprint 2015

Committed.

  • slashrsm committed ff4e955 on 8.x-1.x authored by Wim Leers
    Issue #2628150 by Wim Leers: Update README & .info.yml description
    

  • Devin Carlson committed 79b0f2f on 7.x-3.x
    Issue #2628150 by Wim Leers: Update README & .info description
    

  • Devin Carlson committed 15f5718 on 7.x-2.x
    Issue #2628150 by Wim Leers: Update README & .info description
    

  • Devin Carlson committed 5555abd on 7.x-1.x
    Issue #2628150 by Wim Leers: Update README & .info description
    

Status: Fixed » Closed (fixed)

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