Problem/Motivation

It is difficult to find real world examples of Drupal's apis in use.

Proposed resolution

Link to the examples project from README.txt

Remaining tasks

Port Examples to D8
Link to the examples project from README.txt

User interface changes

A link will be added to the examples project from README.txt

API changes

none

Original report by @cweagans

CommentFileSizeAuthor
#8 2072043_08.patch355 bytesfrob
#6 2072043_06.patch350 byteser.pushpinderrana
#1 2072043_01.patch337 bytescweagans
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cweagans’s picture

Status: Active » Needs review
FileSize
337 bytes
jhodgdon’s picture

IMO it is a bit premature to add this to the D8 README, especially since there is an effort underway to move the most crucial examples into core, and secondly because so few of the examples have been ported to Drupal 8 so it's not all that useful yet.

DamienMcKenna’s picture

Status: Needs review » Postponed

Per the discussion on #1532612: Move Examples Project into Drupal core, several of the simple examples are going to be added to core, so this may not be needed at all.

frob’s picture

Issue summary: View changes
Status: Postponed » Active
Parent issue: » #1880976: [meta] Port examples (including submodules) to D9.4+

Updated the summery using template. Marking as active because I do think more discussion should happen about this.

Also, from reading the comments it is still up in the air if any of examples will be moving into core

several of the simple examples are going to be added to core

and even if some of it did we should still have a link to the examples project for the non-simple examples.

This does cause one issue. There will be a great deal more pressure on the examples project to be kept up-to-date and not allow code in that doesn't follow proper coding standards.

Mile23’s picture

Status: Active » Needs work

It doesn't seem that moving Examples into core is in the cards at this point. But even if it is moved into core, it will be piecemeal over time, and thus a link to the external project would be appreciated.

+++ b/README.txt
@@ -114,3 +114,5 @@ More about developing:
+ * Learn from contributed code examples:

I'd say "Learn from documented Drupal API examples:"

er.pushpinderrana’s picture

Status: Needs work » Needs review
FileSize
350 bytes

Updated the patch as per #5 comment.

jhodgdon’s picture

Status: Needs review » Needs work

This patch doesn't seem to match the suggestion in #5... I think the wording is better in #5. Thanks!

frob’s picture

FileSize
355 bytes

hows this?

all i did was change the words.

frob’s picture

Status: Needs work » Needs review
Mile23’s picture

Status: Needs review » Reviewed & tested by the community

Looks great! Thanks, @frob.

jhodgdon’s picture

Status: Reviewed & tested by the community » Fixed

I was just going to commit this but... should be https://www.drupal.org right?

Fixed that on commit. Thanks!

  • jhodgdon committed 165e9e9 on 8.0.x
    Issue #2072043 by frob, er.pushpinderrana, Mile23, cweagans: Add link to...

Status: Fixed » Closed (fixed)

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

ressa’s picture

Sadly, it looks like this link was lost during an update of the README file ... Perhaps it can be added again?

https://git.drupalcode.org/project/drupal/-/blob/11.x/README.md